Skip to content

fix: shorten Exchange calendar app display names to avoid being truncated in onboarding v3 flow#28075

Open
romitg2 wants to merge 2 commits intocalcom:mainfrom
romitg2:fix/shorten-exchange-cal-names
Open

fix: shorten Exchange calendar app display names to avoid being truncated in onboarding v3 flow#28075
romitg2 wants to merge 2 commits intocalcom:mainfrom
romitg2:fix/shorten-exchange-cal-names

Conversation

@romitg2
Copy link
Member

@romitg2 romitg2 commented Feb 19, 2026

before:

Issue: in onboarding v3 flow for exchange 2013, and 2016 both only Microsoft calendar ... is visible, unable to see which one '13, and which one is '16.

Screenshot 2026-02-19 at 8 57 59 PM Screenshot 2026-02-19 at 8 51 52 PM

after:

Screenshot 2026-02-19 at 8 50 54 PM Screenshot 2026-02-19 at 5 00 11 PM
  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

  • Are there environment variables that should be set?
  • What are the minimal test data to have?
  • What is expected (happy path) to have (input and output)?
  • Any other important info that could help to test that PR

Checklist

  • I haven't read the contributing guide
  • My code doesn't follow the style guidelines of this project
  • I haven't commented my code, particularly in hard-to-understand areas
  • I haven't checked if my changes generate no new warnings
  • My PR is too large (>500 lines or >10 files) and should be split into smaller PRs

Summary by cubic

Shortened display names for the Exchange 2013 and Exchange 2016 calendar apps to prevent truncation in the onboarding v3 flow. Updated metadata name and title to "Exchange 2013" and "Exchange 2016".

Written for commit d10fe7e. Summary will update on new commits.

@romitg2 romitg2 requested a review from a team as a code owner February 19, 2026 16:40
@graphite-app graphite-app bot added the community Created by Linear-GitHub Sync label Feb 19, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 19, 2026

Hey there and thank you for opening this pull request! 👋🏼

We require pull request titles to follow the Conventional Commits specification and it looks like your proposed title needs to be adjusted.

Details:

No release type found in pull request title " fix: shorten Exchange calendar app display names to avoid being truncated in onboarding v3 flow". Add a prefix to indicate what kind of release this pull request corresponds to. For reference, see https://www.conventionalcommits.org/

Available types:
 - feat: A new feature
 - fix: A bug fix
 - docs: Documentation only changes
 - style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
 - refactor: A code change that neither fixes a bug nor adds a feature
 - perf: A code change that improves performance
 - test: Adding missing tests or correcting existing tests
 - build: Changes that affect the build system or external dependencies (example scopes: gulp, broccoli, npm)
 - ci: Changes to our CI configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs)
 - chore: Other changes that don't modify src or test files
 - revert: Reverts a previous commit

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 issue found across 2 files

Prompt for AI agents (all issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="packages/app-store/exchange2013calendar/_metadata.ts">

<violation number="1" location="packages/app-store/exchange2013calendar/_metadata.ts:4">
P2: User-facing app display strings should go through i18n (t()) and a translation key instead of hardcoded literals. Add a locale key for the app name and use t(...) here to avoid unlocalized text.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

@romitg2 romitg2 requested a review from anikdhabal February 19, 2026 16:49
@romitg2 romitg2 added ready-for-e2e run-ci Approve CI to run for external contributors labels Feb 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community Created by Linear-GitHub Sync ready-for-e2e run-ci Approve CI to run for external contributors size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments