Skip to content

Add Brazilian Portuguese (pt-BR) language support - #70

Open
plgs2005 wants to merge 2 commits into
fspecii:mainfrom
plgs2005:feature/pt-br-language-support
Open

Add Brazilian Portuguese (pt-BR) language support#70
plgs2005 wants to merge 2 commits into
fspecii:mainfrom
plgs2005:feature/pt-br-language-support

Conversation

@plgs2005

Copy link
Copy Markdown

Summary

Adds Brazilian Portuguese (pt-BR) localization support to ACE-Step UI.

Included

  • Added pt-BR translations
  • Added Portuguese to language selector
  • Improved locale date formatting
  • Better accessibility for Portuguese-speaking users

Why

Brazil has a huge creator and music community. This helps expand ACE-Step UI adoption in LATAM.

Copilot AI review requested due to automatic review settings April 27, 2026 18:54

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 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".

Comment thread context/I18nContext.tsx Outdated
Comment thread components/SettingsModal.tsx

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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 pt language to the i18n Language union 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.

Comment thread context/I18nContext.tsx Outdated
Comment thread i18n/translations.ts
Comment thread components/SettingsModal.tsx
Comment thread components/SettingsModal.tsx

@plgs2005 plgs2005 left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Fixed in latest commit. Added missing Portuguese translation keys and restored English as default fallback.

@plgs2005 plgs2005 left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Fixed in latest commit. Added missing Portuguese translation keys and restored English as default fallback.

@plgs2005 plgs2005 left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Fixed in latest commit. Added missing Portuguese translation keys and restored English as default fallback.

@plgs2005

plgs2005 commented May 5, 2026

Copy link
Copy Markdown
Author

All review comments have been addressed.

  • Restored English fallback
  • Completed missing translations
  • Cleaned up code

Ready for merge 🚀

@plgs2005

plgs2005 commented May 7, 2026

Copy link
Copy Markdown
Author

Hi! @fspecii

Just following up on PR #70 regarding the Brazilian Portuguese localization support.
All requested review changes were addressed and the PR was approved by review.
Ready for final maintainer review / merge whenever possible... tks

Thanks for the awesome project!

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.

3 participants