-
Notifications
You must be signed in to change notification settings - Fork 910
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
[iOS] Timer on Sync QR Code and Phrase Words screens is not translated #25107
Conversation
[puLL-Merge] - brave/brave-core@25107 DescriptionThis PR reorganizes and updates the localization strings for the Sync feature in the Brave iOS app. It moves all Sync-related strings into a dedicated ChangesChanges
This refactoring improves code organization by grouping all Sync-related strings together and makes it easier to maintain and update these strings in the future. |
A Storybook has been deployed to preview UI for the latest push |
Resolves brave/brave-browser#40051
In addition to adding missing translation for timer countdown, this Pr is also organizing the Sync localization keys and separating them to a separate file. This will help for future decoupling Sync as a library
Submitter Checklist:
QA/Yes
orQA/No
;release-notes/include
orrelease-notes/exclude
;OS/...
) to the associated issuenpm run test -- brave_browser_tests
,npm run test -- brave_unit_tests
wikinpm run presubmit
wiki,npm run gn_check
,npm run tslint
git rebase master
(if needed)Reviewer Checklist:
gn
After-merge Checklist:
changes has landed on
Test Plan:
Simulator.Screen.Recording.-.iPhone.15.Pro.-.2024-08-13.at.14.19.34.mp4
Set device language to non-English
Launch Brave
Settings > Sync
Create a new Sync chain
Add device > Observe