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 Privacy Pro translations #3700

Merged
merged 8 commits into from
Dec 10, 2024
Merged

Conversation

miasma13
Copy link
Contributor

@miasma13 miasma13 commented Dec 9, 2024

Task/Issue URL: https://app.asana.com/0/0/1208750072186506/f
CC: @bwaresiak

Description:
Update Privacy Pro and VPN related translations.

Steps to test this PR:
Change preferred language for the app and verify completeness for:

  • Privacy Pro settings when not signed in
  • Privacy Pro settings when signed in
  • VPN settings and popover views

Definition of Done (Internal Only):

Copy Testing:

  • Use of correct apostrophes in new copy, ie rather than '

Orientation Testing:

  • Portrait
  • Landscape

Device Testing:

  • iPhone SE (1st Gen)
  • iPhone 8
  • iPhone X
  • iPhone 14 Pro
  • iPad

OS Testing:

  • iOS 15
  • iOS 16
  • iOS 17

Theme Testing:

  • Light theme
  • Dark theme

Internal references:

Software Engineering Expectations
Technical Design Template

@miasma13 miasma13 requested a review from samsymons December 9, 2024 18:13
Copy link

github-actions bot commented Dec 9, 2024

Warnings
⚠️ PR has more than 500 lines of code changing. Consider splitting into smaller PRs if possible.
Messages
📖

You seem to be updating localized strings. Make sure that you request translations and include translated strings before you ship your change. See Localization Guidelines for more information.

Generated by 🚫 dangerJS against cc32253

Comment on lines 1083 to 1080
"duckplayer.presentation.modal.body" = "Duck Player giver dig mulighed for at se YouTube uden målrettede annoncer i en biograflignende oplevelse i DuckDuckGo, og det, du ser, påvirker ikke dine anbefalinger.";
"duckplayer.presentation.modal.body" = "Duck Player lets you watch YouTube without targeted ads in DuckDuckGo and what you watch won’t influence your recommendations.";
Copy link
Contributor

Choose a reason for hiding this comment

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

This change seems incorrect?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was one of two excluded duckplayer strings and apparently it also changed the value. Removed exclusion and re-imported the strings, should be fine now.

"duckplayer.settings.info-text" = "Duck Player giver en ren seeroplevelse uden målrettede annoncer og forhindrer, at visningsaktivitet påvirker dine YouTube-anbefalinger.";
"duckplayer.settings.info-text" = "Duck Player lets you watch YouTube without targeted ads in DuckDuckGo and what you watch won’t influence your recommendations.";
Copy link
Contributor

Choose a reason for hiding this comment

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

This one looks incorrect too.

Comment on lines -1535 to -1536
/* No comment provided by engineer. */
"LOREM IPSUM" = "LOREM IPSUM";
Copy link
Contributor

Choose a reason for hiding this comment

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

🫡

Copy link
Contributor

@samsymons samsymons left a comment

Choose a reason for hiding this comment

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

Approving, though with a comment about some Duck Player strings, looks like some of these have reverted to English. Otherwise, I understand that the remaining English strings like VPN tips will get translations in a separate PR.

Simulator Screenshot - iPhone 16 Pro Max - 2024-12-09 at 18 07 13

@miasma13 miasma13 merged commit 26c15ea into main Dec 10, 2024
13 checks passed
@miasma13 miasma13 deleted the michal/update-pp-translations branch December 10, 2024 08:45
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.

2 participants