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 #3620

Closed
wants to merge 19 commits into from

Conversation

miasma13
Copy link
Contributor

@miasma13 miasma13 commented Dec 2, 2024

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

Description:
Update Privacy Pro and VPN related translations.

Optional E2E tests:

  • Run PIR E2E tests
    Check this to run the Personal Information Removal end to end tests. If updating CCF, or any PIR related code, tick this.

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 references:

Pull Request Review Checklist
Software Engineering Expectations
Technical Design Template
Pull Request Documentation

Copy link
Contributor

github-actions bot commented Dec 2, 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 f1cf08f

@miasma13 miasma13 force-pushed the michal/update-pp-translations branch from e67408e to 7a4a930 Compare December 6, 2024 17:32
Copy link
Collaborator

@SabrinaTardio SabrinaTardio left a comment

Choose a reason for hiding this comment

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

For new Submodules, (and potentially targets) add English US to the list of languages and translate one string.
For SyncUI we noticed that english was defaulting to another language and doing this fixed it

Some String from a catalogue and some full catalogues are still missing translation but I assume this are still ongoing

DuckDuckGo/Localizable.xcstrings Show resolved Hide resolved
@daxmobile daxmobile deleted the branch release/1.117.0 December 9, 2024 13:09
@daxmobile daxmobile closed this Dec 9, 2024
@miasma13
Copy link
Contributor Author

PR closed due to old release branch being deleted. Reopened in #3656

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.

5 participants