Skip to content

feat(i18n): Adding pt-BR translations#1157

Merged
adamshiervani merged 2 commits into
jetkvm:devfrom
rubenskj:feature/adding-pt-localization
Jan 27, 2026
Merged

feat(i18n): Adding pt-BR translations#1157
adamshiervani merged 2 commits into
jetkvm:devfrom
rubenskj:feature/adding-pt-localization

Conversation

@rubenskj
Copy link
Copy Markdown
Contributor

@rubenskj rubenskj commented Jan 24, 2026

Summary

This PR adds Portuguese (Brazil) localization to the JetKVM UI. Created ui/localization/messages/pt.json with all translated strings and updated the inlang settings to register the new pt locale.

Checklist

  • Ran make test_e2e locally and passed
  • Linked to issue(s) above by issue number (e.g. Closes #<issue-number>)
  • One problem per PR (no unrelated changes)
  • Lints pass; CI green
  • Tricky parts are commented in code

Tests

image image

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jan 24, 2026

CLA assistant check
All committers have signed the CLA.

@rubenskj
Copy link
Copy Markdown
Contributor Author

I noticed a few strings that are not being translated as intended.

Macro keys

image

Note: The labels Left and Right are still appearing in English.

Developer Settings

image

Note: There is a checkbox label that is not being translated at all. I also couldn’t find any matching key for this text in the {locale}.json files.


Would you prefer that I open a separate issue to track this?

@adamshiervani
Copy link
Copy Markdown
Contributor

Thanks for your the PR!

Would you prefer that I open a separate issue to track this?
That would be awesome.

@adamshiervani adamshiervani merged commit e61567c into jetkvm:dev Jan 27, 2026
2 checks passed
@rubenskj
Copy link
Copy Markdown
Contributor Author

Hi @adamshiervani, thanks for the reply!

I’ve opened an issue describing the missing translations and where they can be fixed:
#1162

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