Closed
Bug 1347219
Opened 7 years ago
Closed 7 years ago
Enable Pontoon for Firefox Screenshots system add-on (excluding website)
Categories
(Localization Infrastructure and Tools :: Administration / Setup, task)
Localization Infrastructure and Tools
Administration / Setup
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jhirsch, Assigned: flod)
References
Details
This bug covers localization of just the strings that will ship in the Page Shot system addon. We'll handle the website in a separate bug. Answering the questions listed on the Implementing Pontoon MDN page[1]: - Project name: Page Shot (the name is not final) - Repository URL: https://github.com/mozilla-services/pageshot/ - List of requested locales: I believe we want all the locales, since this will be shipping in Firefox as a system addon. - Website URL: The website isn't yet localized, and the final domain name choice is blocked on the unresolved naming choice. [1] https://mdn.io/Implementing_Pontoon_in_a_Mozilla_website#C._Add_your_project_to_Pontoon
Reporter | ||
Comment 1•7 years ago
|
||
This bug is tracked in the page shot github repo: https://github.com/mozilla-services/pageshot/issues/2295
Assignee | ||
Comment 2•7 years ago
|
||
Completely missed this bug (was looking at bug 1344898). I have a few questions: * Is the website going to live in the same repository, or a separate one? * Are the strings stable and ready to be exposed? For example you're talking about the name not being final. If that's the case, I'm not sure it makes sense to expose them now. * Once finalized, can the name be localized or transliterated? * Do you prefer localization to commit in a separate repository for l10n (like Activity Stream), commit to master (like Test Pilot), or commit to a specific branch (like Snooze Tabs)? Personally I'd prefer the l10n separate repository, so localizers can commit directly if they prefer to.
Reporter | ||
Updated•7 years ago
|
Flags: needinfo?(6a68)
Summary: Enable Pontoon for Page Shot add-on (excluding website) → Enable Pontoon for Firefox Screenshots system add-on (excluding website)
Reporter | ||
Comment 3•7 years ago
|
||
Hi Francesco, > * Is the website going to live in the same repository, or a separate one? The same repository, although now we are likely to rename that repository. > * Are the strings stable and ready to be exposed? For example you're talking about the name not being final. If that's the case, I'm not sure it makes sense to expose them now. We just got a final name, and product is working on getting final approval for the addon strings right now. I can needinfo you when those are finalized, should be later this week. Note that we've been planning to delay extracting the strings for the website, because the 54 uplift deadline is fast approaching. The website strings would be a separate properties file in the same repo. > * Once finalized, can the name be localized or transliterated? Good question, and still an open question. ni? jgruen to update this bug once the question is settled. > * Do you prefer localization to commit in a separate repository for l10n (like Activity Stream), commit to master (like Test Pilot), or commit to a specific branch (like Snooze Tabs)? I've already got it wired up to work like Test Pilot, with mozilla-pontoon pushing to master. I'd prefer to just leave it as-is, since the work is done.
Flags: needinfo?(jgruen)
Reporter | ||
Comment 4•7 years ago
|
||
Targeting next week for final strings, per John in a meeting earlier today. The renaming is going to happen immediately as well. I'll ni? :flod when both those blockers are cleared.
Flags: needinfo?(jgruen)
Reporter | ||
Comment 5•7 years ago
|
||
Hi :flod, The final strings have landed in the repo and we're ready for localizers to get to work. ----- A few additional final updates to close out this bug: - Final project name: Firefox Screenshots - Updated Repository URL: https://github.com/mozilla-services/screenshots/ Addressing the earlier question: > * Once finalized, can the name be localized or transliterated? :Flod had the great idea to ask our localizers if we should translate the product name or not, and the results seemed pretty evenly split for and against translating. Product decided (1) not to translate the product name, and also (2) to replace 'Firefox Screenshots' with the localizable 'screenshots' wherever possible. This mirrors the approach taken with Firefox Accounts.
Flags: needinfo?(francesco.lodolo)
Assignee | ||
Comment 6•7 years ago
|
||
Tested integration with Pontoon and it works as expected https://github.com/mozilla-services/screenshots/commit/364c30d71a56dfbc496fb0bf5cd6e7649eced4e2 I should be able to set it on production tomorrow, and send out a communication to localizers. Leaving the NI open until that's done.
Assignee | ||
Comment 7•7 years ago
|
||
Side question in the meantime: is there going to be a way to test the add-on, without building it from source (e.g. dev builds)? Do we have an official short description of this project that I can re-use in the project's info on Pontoon?
Flags: needinfo?(ohai)
Reporter | ||
Comment 8•7 years ago
|
||
> Side question in the meantime: is there going to be a way to test the add-on, without building it from source (e.g. dev builds)? You can install a dev build from this URL: https://pageshot.dev.mozaws.net/homepage/install-test-local.html I'm not sure if the dev URL might change to reflect the new name. If it does, I'll update the URL in this bug. > Do we have an official short description of this project that I can re-use in the project's info on Pontoon? How about: "Take clips and screenshots from the Web and save them temporarily or permanently"? This is the addon description string, which has had lots of product scrutiny. Feel free to tweak as you see fit.
Flags: needinfo?(ohai)
Assignee | ||
Comment 9•7 years ago
|
||
Project is enabled https://pontoon.mozilla.org/projects/firefox-screenshots/ https://l10n.mozilla-community.org/webstatus/?product=firefox-screenshots How often is the test add-on updated?
Flags: needinfo?(francesco.lodolo)
Reporter | ||
Comment 10•7 years ago
|
||
I believe the test add-on should be updated whenever we push to dev. ni? Ian to make sure that happens with each push to master.
Flags: needinfo?(ianb)
Comment 11•7 years ago
|
||
We are updating the dev site (https://pageshot.dev.mozaws.net) on an ad hoc basis. Since ops is revisiting the deployment I'll use this as a chance to get per-commit deployment going; I've filed [#2521](https://github.com/mozilla-services/screenshots/issues/2521) for this. I updated this morning and I'll try to manually update regularly as these translations are coming in: http://pageshot.dev.mozaws.net/homepage/install-test-local.html To see what's installed at any moment there's http://pageshot.dev.mozaws.net/__version__ – there's also an unincludedCommits key there that gives a link to the commits that aren't yet deployed (e.g., right now: https://github.com/mozilla-services/screenshots/compare/d0a5290...master)
Flags: needinfo?(ianb)
Comment 12•7 years ago
|
||
Localization has started, Pontoon is enabled. Closing.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•