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 l10n XTB and messages.json for 76.0.3809.87 #3074

Merged
merged 2 commits into from
Aug 2, 2019

Conversation

mkarolin
Copy link
Collaborator

@mkarolin mkarolin commented Aug 1, 2019

Fixes brave/brave-browser#5500

Submitter Checklist:

Test Plan:

N/A

Reviewer Checklist:

  • New files have MPL-2.0 license header.
  • Request a security/privacy review as needed.
  • Adequate test coverage exists to prevent regressions
  • Verify test plan is specified in PR before merging to source

After-merge Checklist:

  • The associated issue milestone is set to the smallest version that the
    changes has landed on.
  • All relevant documentation has been updated.

@mkarolin mkarolin added the l10n label Aug 1, 2019
@mkarolin mkarolin added this to the 0.70.x - Nightly milestone Aug 1, 2019
@mkarolin mkarolin self-assigned this Aug 1, 2019
@mkarolin mkarolin force-pushed the 76.0.3809.87_localization branch 2 times, most recently from 6a16cd0 to 6fc90d4 Compare August 1, 2019 22:27
In each .xtb file we create, strip all children from <ph> tags and
remove innter text if it start with $ (so that $1 text would be removed,
but something like '&lt;a href="$1"...' would not. This is needed so
that multiple replacement placeholders aren't generated by the resource
data pack loader.
Copy link
Member

@bsclifton bsclifton left a comment

Choose a reason for hiding this comment

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

++

@bsclifton bsclifton merged commit edef5bd into master Aug 2, 2019
@bsclifton bsclifton deleted the 76.0.3809.87_localization branch August 2, 2019 07:37
@bsclifton
Copy link
Member

@mkarolin whoops - just noticed the WIP in the title... but I think this was ready for approval, right?

If so, can you create uplifts? 😄

@mkarolin
Copy link
Collaborator Author

mkarolin commented Aug 2, 2019

@bsclifton, will do. WIP was just in case there were comments on the Transifex script changes.

@mkarolin mkarolin changed the title WIP: Update l10n XTB and messages.json for 76.0.3809.87 Update l10n XTB and messages.json for 76.0.3809.87 Aug 2, 2019
mkarolin pushed a commit that referenced this pull request Aug 2, 2019
Update l10n XTB and messages.json for 76.0.3809.87
mkarolin pushed a commit that referenced this pull request Aug 2, 2019
Update l10n XTB and messages.json for 76.0.3809.87
mkarolin pushed a commit that referenced this pull request Aug 2, 2019
Update l10n XTB and messages.json for 76.0.3809.87
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 this pull request may close these issues.

Update localization for Chromium 76.0.3809.87
2 participants