Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update translation tags, add Portuguese + Brazilian #160

Merged
merged 18 commits into from Aug 1, 2023

Conversation

JSKitty
Copy link
Member

@JSKitty JSKitty commented Jul 23, 2023

Abstract

This PR updates the majority of the app UI (HTML) with i18n translation tags, to catch-up from the ever-growing i18n tech-debt of UI PRs missing i18n data tags, additionally, this PR should prevent services like Google Translate from translating the MPW Seed Phrase; which can lead to loss of funds, and luckily was reported by our Portuguese ambassador.

  • A huge amount of i18n-data tags added, removed and updated. (but not ALL).
  • Added notranslate tags to Seed Phrase to prevent translating them for anyone using Google Translate.
  • Added the finished translation files from our Portuguese ambassador and a Brazilian Labs Collaborator.
  • Some minor UI text simplification, like changing "Backup your wallet" to "Secure your wallet", etc.

This PR is aimed to quickly catch-up on the majority of i18n tagging and template backlog, but NOT to fully translate any soon-to-change UI elements - especially given the Masternodes page is likely to change given #143 and other Redesigns over there.

@JSKitty JSKitty added the Enhancement New feature or request label Jul 23, 2023
@JSKitty JSKitty requested review from Liquid369, Duddino, BreadJS and a team July 23, 2023 16:15
@JSKitty JSKitty self-assigned this Jul 23, 2023
Duddino
Duddino previously approved these changes Jul 24, 2023
Copy link
Member

@Duddino Duddino left a comment

Choose a reason for hiding this comment

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

utACK, I did go through MPW and it seemed to be properly translated

@JSKitty JSKitty requested a review from Duddino July 27, 2023 13:24
@JSKitty
Copy link
Member Author

JSKitty commented Jul 27, 2023

Now good for review, final PT-PT and i18n frameworking finished.

This is easy to check/verify, since it is the Portuguese translaton file with minor edits for the Brazilian variant, run a git-diff between the files to see.
@JSKitty JSKitty changed the title Update translation tags, add Portuguese template Update translation tags, add Portuguese + Brazilian Jul 30, 2023
@JSKitty
Copy link
Member Author

JSKitty commented Aug 1, 2023

This PR will bypass Branch Protection in order to keep MPW moving at necessary pace.

I take responsibility if something breaks. 😉

@JSKitty JSKitty merged commit 7698b74 into master Aug 1, 2023
5 checks passed
@JSKitty JSKitty mentioned this pull request Aug 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants