Skip to content

Team collaboration (4/6): web i18n keys - #5282

Closed
lefarcen wants to merge 1 commit into
feat/collab-daemon-resourcesfrom
feat/collab-web-i18n
Closed

Team collaboration (4/6): web i18n keys#5282
lefarcen wants to merge 1 commit into
feat/collab-daemon-resourcesfrom
feat/collab-web-i18n

Conversation

@lefarcen

@lefarcen lefarcen commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Stacks on (3/6). Split out first so the UI PRs carry only their components.

Adds the translation keys the team-collaboration UI uses — share-to-team, the read-only notice, the workspace switcher, and the cloud sign-in callout — to the typed dictionary and all locale files.

Surface area

  • i18n keys — added across all locales

Adds the translation keys the team-collaboration UI uses — share-to-team, the
read-only notice, the workspace switcher, and the cloud sign-in callout — to the
typed dictionary and all locale files. Split out first so the UI PRs that follow
carry only their components.
@lefarcen
lefarcen requested a review from mrcfps July 8, 2026 03:51
@lefarcen lefarcen added size/M PR changes 100-300 lines risk/low Low risk: docs/i18n/assets only type/feature New feature labels Jul 8, 2026
@lefarcen

lefarcen commented Jul 8, 2026

Copy link
Copy Markdown
Contributor Author

Quick PR-body pass: splitting the i18n layer out ahead of the UI PRs makes the stack easier to review.

Could you add the missing Why, What users will see, and Validation context? The surface-area section is already there, so this should just be a short reshape of the current summary plus what you ran.

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Visual regression review

Head: a9bffbb · Base: a8cd808

Baseline unavailable; PR screenshots are new visual cases and need baseline review.

0 changed · 0 unchanged · 53 new without baseline · 0 failed

New cases without baselines

PR PR PR
visual-avatar-local-agent-list
pr
visual-avatar-local-agent-list-panel
pr
visual-avatar-menu
pr
visual-avatar-menu-panel
pr
visual-avatar-open-design-account
pr
visual-critical-settings
pr
visual-critical-workspace
pr
visual-critical-workspace-preview
pr
visual-design-system-detail
pr
visual-design-systems
pr
visual-home
pr
visual-home-catalog
pr
visual-home-context-picker
pr
visual-home-context-picker-popover
pr
visual-home-plugin-filter
pr
visual-home-plugin-use-staged
pr
visual-home-plugin-use-with-query
pr
visual-home-staged-attachment
pr
visual-integrations
pr
visual-integrations-mcp
pr

33 additional new case(s) omitted from this comment.

Visual diff is advisory only and does not block merging.

@mrcfps mrcfps left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@lefarcen I reviewed the changed i18n ranges across Dict and all registered locale dictionaries. The new team-collaboration keys are consistently added in every locale file, the typed dictionary is updated in the matching groups, and the dependency-free key-presence check passed for all 19 locale files. I also attempted the focused web typecheck, but this Looper worktree has no node_modules, so it fails before reaching these changes due missing workspace dependencies. Thanks for splitting this out cleanly ahead of the UI work; it keeps the stacked review focused.

🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.

@lefarcen

lefarcen commented Jul 8, 2026

Copy link
Copy Markdown
Contributor Author

Rolled into the (2/3) web PR #5283 — i18n now reviewed with the in-project UI.

@lefarcen lefarcen closed this Jul 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk/low Low risk: docs/i18n/assets only size/M PR changes 100-300 lines type/feature New feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants