Skip to content

feat(frontend): i18n boilerplate#573

Open
DarianM wants to merge 3 commits intomainfrom
i18n_boilerplate
Open

feat(frontend): i18n boilerplate#573
DarianM wants to merge 3 commits intomainfrom
i18n_boilerplate

Conversation

@DarianM
Copy link
Copy Markdown
Member

@DarianM DarianM commented Feb 27, 2026

Part of #482

@DarianM DarianM requested a review from sidvishnoi February 27, 2026 12:20
@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 27, 2026

Deployment results

Worker Alias URL Outcome
API - staging success
CDN - staging success
App - 0801d45a success

Logs #23189903025

Comment thread frontend/app/components/offerwall/HowItWorks.tsx Outdated
@DarianM DarianM requested a review from kjmitchelljr March 17, 2026 10:33
Copy link
Copy Markdown
Collaborator

@kjmitchelljr kjmitchelljr left a comment

Choose a reason for hiding this comment

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

Small nits here and there, but overall the PR looks good

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Are there other locales to be added here in the short-term or will his be built out in the future?

"status": {
"connect": "If you're connecting your wallet address for the first time, you'll start with the default profile. You can then customize and save your profile as needed.",
"no_saved_profiles": "There are no custom edits for the {{toolName}} correlated to this wallet address but you can start customizing when you want.",
"profiles_fetched": "We've loaded your profiles. Feel free to keep customizing your {{toolName}} to fit your style."
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

nit: curious reasoning for snake_case here versus camelCase


export function useI18n() {
return useContext(I18nContext)
}
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

nit: adding I18nContext.displayName = 'I18nContext for debugging

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants