Closed Bug 565165 Opened 10 years ago Closed 5 years ago
make required pieces of services
.m .c localizable
Axel, this is something like six strings https://auth.services.mozilla.com/weave-password-reset https://auth.services.mozilla.com/weave-delete-account I'm going to move these onto services somehow, but we'll need strings. Even if they're just six strings in a text file, I can do some hacky stuff to use strings in PHP.
Oops... didn't CC Axel here.
I'd more lean towards putting them into traditional PO and on https://localize.mozilla.org/. CCing folks in the know there.
The goal is that they would live on services.mozilla.com in the future, and we don't necessarily want to localize all of that content, i.e. the changelogs. Not sure of the best path there, but you guys know this space better than I do. :)
https://services.mozilla.com/reset-password/ is up, I'll get the delete account page into SVN as well. Axel, how is best to get this UI localized? I'm a total newb, happy to read docs.
Summary: get delete account/reset password pages localized → get delete account/reset password pages to services.m.c and localized
Status: NEW → ASSIGNED
Summary: get delete account/reset password pages to services.m.c and localized → get required pieces of services.m.c localized
(In reply to comment #4) > https://services.mozilla.com/reset-password/ is up, I'll get the delete account > page into SVN as well. Axel, how is best to get this UI localized? I'm a > total newb, happy to read docs. Mike, where is the code of this page located? I wanted to take a look and see what l10n approach would be the best. For now, without any additional info, I'm with Axel on using Gettext and putting the strings up on Verbatim (localize.mozilla.org). Also, if you don't mind, I'll assign this to myself and change the description. We'll file separate bugs to get this localized, but for now, we need to make it localizable :)
Assignee: mconnor → stas
Summary: get required pieces of services.m.c localized → make required pieces of services.m.c localizable
Hey Stas, thanks for picking this up! The pieces that need l10n support: /delete-account/index.php /reset-password/index.php /privacy-policy/index.html /tos/index.html /index.html /sync is just relnotes/updated pages for the addon, I don't know if we care about localization for those. My initial put is "not really" since the add-on is mostly going to be a leading development channel.
Comment on attachment 467446 [details] [diff] [review] Patch 1. Add gettext support I have installed the site locally with your patch and it works fine, a couple nitpicks though: * your redirection will probably not bypass the caching system without a vary:* header, you can reuse this: http://viewvc.svn.mozilla.org/vc/projects/mozilla-europe.org/trunk/includes/redirect.php?revision=66823&view=markup#l36 another nit in this method is that the Location header should in theory have a full url not a relative one. * we should not have hardcoded en-US url strings in the footer
Attachment #467446 - Flags: feedback?(pascalc) → feedback-
Here's a patch addressing Pascal's comments (thanks Pascal!). All localizable strings still need to be wrapped with _(). This patch does not do this.
I changed the links in the footer point to: * http://www.mozilla.com/$locale/about/legal.html * http://www.mozilla.com/$locale/privacy-policy.html but I think that they still redirect to en-US versions. I think we should link to http://mozilla-europe.org/$locale/about/privacy/ for European locales. (Or, even better, redirect http://www.mozilla.com/$locale/privacy-policy.html to http://mozilla-europe.org/$locale/about/privacy/.) Not sure about the Legal Notices page. Do we have translations for that?
Comment on attachment 467815 [details] [diff] [review] Patch 2. Addressed Pascal's comments Looks good to me. I can handle getting the strings hooked up after this lands.
Attachment #467815 - Flags: review?(mconnor) → review+
So... we've replaced the general user-facing parts with Account Portal (https://account.services.mozilla.com). There's the add-on landing page (add-on will be deprecated soon), the front-page, and the ToS/PP. I guess the question is... do we care about what's left here? Nothing actually links to services.m.c directly anymore, except for the PP/ToS...
No longer blocks: 592378
Closing as fixed. This was replaced by Firefox Accounts which is localized.
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.