Closed
Bug 1130565
Opened 10 years ago
Closed 10 years ago
Add video URL/locale relationships to Hello product page
Categories
(www.mozilla.org :: Pages & Content, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ckprice, Assigned: jpetto)
References
()
Details
(Whiteboard: [kb=1665659] )
Attachments
(1 file)
This bug is to add additional video URL's/locales for the Hello brand video to the product page. New locales and URL's are described in bug1119046comment19. This bug is blocked by the production and approval of the additional videos.
It also includes incorporating en-US as the backup for all other locales.
Ref: bug 1129082 in which we discussed other possible options, but ultimately landed on the existing implementation. This bug also includes a script for extraction of fallback locales for possible future use.
Reporter | ||
Updated•10 years ago
|
Whiteboard: [kb=1665659]
Reporter | ||
Comment 1•10 years ago
|
||
:jpetto - bug1119046comment70 is complete. Think you can have done by Thursday? Let me know if we're missing any info for you.
Flags: needinfo?(jon)
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → jon
Flags: needinfo?(jon)
OS: Mac OS X → All
Hardware: x86 → All
Assignee | ||
Comment 2•10 years ago
|
||
Assignee | ||
Comment 3•10 years ago
|
||
:ckprice - Can you confirm the changes made in the PR? New locales listed in bug1119046comment70 have been added, and code has been put in place to serve the en-US video to any locale not specified in the dictionary.
Flags: needinfo?(cprice)
Comment 5•10 years ago
|
||
Commits pushed to master at https://github.com/mozilla/bedrock
https://github.com/mozilla/bedrock/commit/6193a4fbfe0b4879625e82a06982ca237a7a4567
Bug 1130565. Add more localized videos to Hello page.
https://github.com/mozilla/bedrock/commit/37ae22dd8fbc0adbec91ab07f8a3ad053e68b76c
Merge pull request #2746 from jpetto/bug-1130565-add-video-locales-hello-page
Bug 1130565. Add more localized videos to Hello page.
Reporter | ||
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•