Skip to content

Frontend/i18n: Remove inline <Trans> text#9211

Merged
tristanlabelle merged 3 commits into
developfrom
frontend/refactor/remove-inline-trans-strings
Jul 3, 2026
Merged

Frontend/i18n: Remove inline <Trans> text#9211
tristanlabelle merged 3 commits into
developfrom
frontend/refactor/remove-inline-trans-strings

Conversation

@tristanlabelle

@tristanlabelle tristanlabelle commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Our <Trans> tags often included a copy of the string from en.json, which unavoidably lead to the two drifting apart as noted in #9008. It turns out we don't have to do that and can simply provide the components to be instantiated by the string without duplicating the text.

Closes #9008

Testing

Used vercel preview to spot check:

ProfileIncompleteDialog

image

StrongVerificationPage (note the text is from the more recent en.json, the inline text was stale)

image

PushNotificationSetting

image

Web frontend checklist

  • There are no console warnings when running the app
  • Added tests where relevant
  • Clicked around my changes running locally and it works
  • Checked Desktop, Mobile and Tablet screen sizes

For maintainers

  • Maintainers can push commits to my branch
  • Maintainers can merge this PR for me

@CouchersBot

CouchersBot commented Jun 30, 2026

Copy link
Copy Markdown
Member

Mobile

Install the Dev Tool (iOS via TestFlight, iOS Simulator, or Android .apk) here.

Scan the QR with your phone camera, or tap Open in Dev Tool on the device, to open this branch in the installed Dev Tool dev client.

    iOS         Android    
    QR to open the iOS build         QR to open the Android build    
    Open in Dev Tool         Open in Dev Tool    
Deep links

iOS

couchers-devtool://expo-development-client/?url=https%3A%2F%2F16364614--ota.preview.couchershq.org%2Fios%2Fmanifest

Android

couchers-devtool://expo-development-client/?url=https%3A%2F%2F16364614--ota.preview.couchershq.org%2Fandroid%2Fmanifest

Web (Vercel)

View the Vercel web preview for this branch.

Backend

Schema Schema diff Sample emails

Other

Protos Backend coverage Web coverage

@tristanlabelle
tristanlabelle marked this pull request as ready for review July 1, 2026 13:36
@tristanlabelle
tristanlabelle requested a review from nabramow as a code owner July 1, 2026 13:36
@tristanlabelle
tristanlabelle requested a review from a team July 1, 2026 13:36
@tristanlabelle tristanlabelle added the i18n Internationalization / localization issue label Jul 1, 2026

@nabramow nabramow left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Oh great, I always thought this seemed a bit weird! I tested a couple and seems okay to me!

@tristanlabelle
tristanlabelle merged commit 52d3c22 into develop Jul 3, 2026
8 checks passed
@tristanlabelle
tristanlabelle deleted the frontend/refactor/remove-inline-trans-strings branch July 3, 2026 12:03
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

📝 Release Notes

This PR does not need to be included in release notes.

Reason: This PR is primarily an internal i18n/code cleanup that removes duplicated inline translation text and aligns the frontend with existing translation strings. While it may fix a few stale wording mismatches, it does not introduce a new user-facing feature or a significant end-user-visible bug fix on its own.

🤖 Bot Debug Information

Model: couchers.openai.gpt-5.4
Decision: exclude
Reasoning: This PR is primarily an internal i18n/code cleanup that removes duplicated inline translation text and aligns the frontend with existing translation strings. While it may fix a few stale wording mismatches, it does not introduce a new user-facing feature or a significant end-user-visible bug fix on its own.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

i18n Internationalization / localization issue release notes: not needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Frontend/i18n: Investigate removing duplicated English placeholders

3 participants