Add Brazilian Portuguese (pt-BR) language support - #70
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8633b434c4
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
Pull request overview
Adds Brazilian Portuguese localization support to the ACE-Step UI i18n system and exposes it in Settings, including pt-BR date formatting.
Changes:
- Added
ptlanguage to the i18nLanguageunion and introduced a new Portuguese translation block. - Updated language persistence logic to recognize
pt. - Updated Settings language selector to include Portuguese and improved “member since” date locale formatting via a locale map.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
i18n/translations.ts |
Adds pt language support and Portuguese translation strings. |
context/I18nContext.tsx |
Updates stored-language validation / default selection logic to include pt. |
components/SettingsModal.tsx |
Adds Portuguese to the language dropdown and switches date formatting to use a language→locale map. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
plgs2005
left a comment
There was a problem hiding this comment.
Fixed in latest commit. Added missing Portuguese translation keys and restored English as default fallback.
plgs2005
left a comment
There was a problem hiding this comment.
Fixed in latest commit. Added missing Portuguese translation keys and restored English as default fallback.
plgs2005
left a comment
There was a problem hiding this comment.
Fixed in latest commit. Added missing Portuguese translation keys and restored English as default fallback.
|
All review comments have been addressed.
Ready for merge 🚀 |
Summary
Adds Brazilian Portuguese (pt-BR) localization support to ACE-Step UI.
Included
Why
Brazil has a huge creator and music community. This helps expand ACE-Step UI adoption in LATAM.