Push legal-docs commits in master to production

RESOLVED FIXED

Status

www.mozilla.org
Bedrock
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: malexis, Assigned: kohei)

Tracking

Development/Staging

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [kb=1629325] )

Attachments

(3 attachments)

(Reporter)

Description

2 years ago
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

Updated

2 years ago
Duplicate of this bug: 1121090
(Assignee)

Comment 2

2 years ago
Created attachment 8548397 [details] [review]
pull request
Assignee: nobody → kohei.yoshino
Status: NEW → ASSIGNED
(Assignee)

Updated

2 years ago
OS: Mac OS X → All
Hardware: x86 → All
Whiteboard: [kb=1629325]

Comment 3

2 years ago
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

Updated

2 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED

Comment 4

2 years ago
How soon can we see the updates? I checked on German links, they all pointed to en-US.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 5

2 years ago
Ah, we have to activate locales manually until Bug 1097785 is fixed... :( Checking it out now.
Status: REOPENED → ASSIGNED
(Assignee)

Comment 6

2 years ago
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.
Flags: needinfo?(francesco.lodolo)
(Assignee)

Comment 7

2 years ago
Created attachment 8548530 [details] [review]
pull request to fix locale names on legal-docs
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.
Flags: needinfo?(francesco.lodolo) → needinfo?(kohei.yoshino)
(Assignee)

Comment 9

2 years ago
> 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...
Flags: needinfo?(kohei.yoshino)
(Assignee)

Comment 10

2 years ago
(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.
(Assignee)

Comment 11

2 years ago
Created attachment 8549073 [details] [review]
pull request for bedrock, take 2
(Assignee)

Comment 12

2 years ago
(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?
Flags: needinfo?(francesco.lodolo)
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.
Flags: needinfo?(francesco.lodolo)
(Assignee)

Comment 14

2 years ago
Thank you! Then we can close this bug.
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.