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

[iOS] Add Transaltion for 1.73 #42622

Closed
nuo-xu opened this issue Dec 2, 2024 · 2 comments · Fixed by brave/brave-core#26835
Closed

[iOS] Add Transaltion for 1.73 #42622

nuo-xu opened this issue Dec 2, 2024 · 2 comments · Fixed by brave/brave-core#26835
Assignees
Labels

Comments

@nuo-xu
Copy link

nuo-xu commented Dec 2, 2024

Test plan:

  1. Set your device to any language that Brave Support
  2. Open Brave Browser App
  3. Observe all copies should be localized correctly

Note: in 1.73 we will introduce support for Hebrew, Thai and Vietnamese. Can be verified in other issue #42181

@nuo-xu nuo-xu added l10n OS/iOS Fixes related to iOS browser functionality QA/Yes release-notes/exclude labels Dec 2, 2024
@nuo-xu nuo-xu self-assigned this Dec 2, 2024
@brave-builds brave-builds added this to the 1.73.x - Release #3 milestone Dec 2, 2024
@kjozwiak
Copy link
Member

kjozwiak commented Dec 2, 2024

Closing the above as brave/brave-core#26814 was merged into the 1.73.x branch. The above requires 1.73.96 or higher for 1.73.x verification 👍

@kjozwiak kjozwiak closed this as completed Dec 2, 2024
@Uni-verse Uni-verse added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Dec 3, 2024
@Uni-verse
Copy link
Contributor

Uni-verse commented Dec 3, 2024

Verified on iPhone 12 running iOS 18.0 using version 1.73 (96)

  • Ensured new and existing strings have not regressed for French, Russian and German locales.

French

example example example example
Image Image Image Image
Image Image Image Image

German

example example example
Image Image Image
Image Image Image
Image Image Image

Russian

example example example example
Image Image Image Image
Image Image Image Image
Image Image Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants