-
Notifications
You must be signed in to change notification settings - Fork 2.7k
feat: rename Account tab to Cloud tab #7558
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- Renamed AccountView component to CloudView - Updated all imports and references from AccountView to CloudView - Changed tab type from 'account' to 'cloud' - Updated command from accountButtonClicked to cloudButtonClicked - Updated icon from to - Updated translation keys from account:* to cloud:* - Renamed all locale files from account.json to cloud.json - Updated package.nls.json command title from 'Account' to 'Cloud' - Updated all test files to use new component names - Kept telemetry event names as ACCOUNT_* for continuity
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I renamed Account to Cloud everywhere except where it counts. Reviewing my own code is like debugging in a mirror.
Issues Found
Critical Issues:
-
Incomplete localization updates - Non-English locale files (es/cloud.json, fr/cloud.json, etc.) still have "title": "Cuenta"/"Compte" instead of "Cloud". These files were renamed but their content wasn't updated.
-
Missing translation key updates - ShareButton.tsx lines 260, 264, 268 still reference old "account:" translation keys:
- Line 260:
{t("account:cloudBenefitSharing")}should be{t("cloud:cloudBenefitSharing")} - Line 264:
{t("account:cloudBenefitHistory")}should be{t("cloud:cloudBenefitHistory")} - Line 268:
{t("account:cloudBenefitMetrics")}should be{t("cloud:cloudBenefitMetrics")}
- Line 260:
Suggestions:
- Consider adding a comment in CloudView.tsx explaining why ACCOUNT_* telemetry events are kept for continuity
These issues will cause runtime errors and need to be fixed before merging.
|
@roomote-agent fix the remaining issues. Switch into translate mode for generating the translations. |
|
I see the issues that need to be fixed:
Working on these fixes now and will switch to translate mode for generating the translations as requested. |
- Fixed ShareButton.tsx to use "cloud:" translation keys instead of "account:" - Updated all non-English locale files to use "Cloud" as title (technical term) - Added comments explaining ACCOUNT_* telemetry naming for backward compatibility
|
✅ All issues have been fixed successfully! Changes made:
Status:
The changes have been pushed to the PR branch and are ready for review. |
Summary
This PR renames the Account tab to Cloud tab throughout the application, as requested via Slack.
Changes Made
Component & UI Changes
AccountViewcomponent toCloudViewcomponents/account/tocomponents/cloud/'account'to'cloud'in App.tsxCommand & Event Changes
accountButtonClickedtocloudButtonClicked$(account)to$(cloud)Localization Updates
account:*tocloud:*account.jsontocloud.jsonTest Updates
Telemetry
ACCOUNT_*for continuity (as requested)Testing
Review Confidence
Code review completed with 92% confidence (High)
Closes Slack request: 'Change the Account tab (AccountView and related entry points) to be the Cloud tab'
Important
Renamed 'Account' tab to 'Cloud' across components, commands, and localization files, updating tests and maintaining telemetry continuity.
AccountViewtoCloudView.components/account/tocomponents/cloud/.'account'to'cloud'inApp.tsx.accountButtonClickedtocloudButtonClicked.package.jsonand types.$(account)to$(cloud).account:*tocloud:*.account.jsontocloud.json.package.nls.jsoncommand title.ACCOUNT_*for continuity (as requested).This description was created by
for 9c8387a. You can customize this summary. It will automatically update as commits are pushed.