1121091 - Push legal-docs commits in master to production

Status: RESOLVED FIXED

(www.mozilla.org :: Bedrock, defect)

Description

[Mike Alexis [:malexis]]

New commits in legal-docs repo related to Hello launch that happened today need to be pushed to prod.

Can we get these live today?

[1] https://github.com/mozilla/legal-docs/tree/master/WebRTC_ToS
[2] https://github.com/mozilla/legal-docs/tree/master/WebRTC_PrivacyNotice
[3] https://github.com/mozilla/legal-docs/tree/master/acceptable_use_policy
[4] https://github.com/mozilla/legal-docs/tree/master/Websites_ToU

Comment 2

[Kohei Yoshino [:kohei]]

Attachment: pull request

Comment 3

[[github robot]]

Commits pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/01d2a90ec252bbebe7a646c9f67c736e816dd7f9
Fix Bug 1121091 - Push legal-docs commits in master to production

https://github.com/mozilla/bedrock/commit/dabf4a01ee1a5e6fbcacd15a261ac957cc9971cb
Merge pull request #2652 from kyoshino/bug-1121091-update-legal-docs

Fix Bug 1121091 - Push legal-docs commits in master to production

Comment 4

[Peiying Mo [:CocoMo]]

How soon can we see the updates? I checked on German links, they all pointed to en-US.

Comment 5

[Kohei Yoshino [:kohei]]

Ah, we have to activate locales manually until Bug 1097785 is fixed... :( Checking it out now.

Comment 6

[Kohei Yoshino [:kohei]]

Added lang files to SVN trunk in r137008. flod: can you double check the commit and push to stage & production?

By the way, es.md and sr-LATN.md on legal-docs don't match locales on Bedrock so those cannot be activated. I'll send a pull request to fix the inconsistency.

Comment 7

[Kohei Yoshino [:kohei]]

Attachment: pull request to fix locale names on legal-docs

Comment 8

[Francesco Lodolo [:flod]]

A bit confused about bn-IN. We're not tracking that locale for Firefox OS material, and those are the only locales we're tracking for /legal right now.

> Added lang files to SVN trunk in r137008. flod: can you double check the
> commit and push to stage & production?

I'm not being smart this morning. Everything looked good so pushed to stage in r137012.
Then realized it's missing a "#" in ## active ##. Fixed dev in r137014, stage in r137015, prod in r137016.

Please note that one thing is adding a file just to enable a page, one other having strings in them. 
Lang files are not tracked automatically in the webdashboard, so localizers simply won't see the missing strings.

Given bug 1097785 for these activations, can we move these strings in legal/index.lang and make the templates use that file? It would also help to have "Back to Legal" read from that file.

Adding this to docs-base.html should be enough
{% add_lang_files "legal/index" %}

I'll take care of moving strings around eventually.

That way, when bug 1097785 is fixed, we can just drop the extra files without loosing strings.

Comment 9

[Kohei Yoshino [:kohei]]

> Then realized it's missing a "#" in ## active ##

Doh, sorry about that bad copy pasta or something :( Hope Bug 1097785 would be solved soon. And yeah, moving strings to index.lang should have avoided additional files...

Comment 10

[Kohei Yoshino [:kohei]]

(In reply to Francesco Lodolo [:flod] from comment #8)
> Adding this to docs-base.html should be enough
> {% add_lang_files "legal/index" %}

es-ES, sr (Comment 7) and ja (Bug 1110467) are still on the way. Once my pull request on legal-docs is merged, I'll update the legal-docs submodule again along with that change.

Comment 11

[Kohei Yoshino [:kohei]]

Attachment: pull request for bedrock, take 2

Comment 12

[Kohei Yoshino [:kohei]]

(In reply to Kohei Yoshino [:kohei] from comment #11)
> Created attachment 8549073 [details] [review]
> pull request for bedrock, take 2

This has been merged today. :flod can you move the strings, including the new es-ES, ja and sr locales?

Comment 13

[Francesco Lodolo [:flod]]

Already copied in r137121

I didn't remove them from the original files, just told localizers to ignore them, hopefully they'll be gone soon.

Comment 14

[Kohei Yoshino [:kohei]]

Thank you! Then we can close this bug.