Closed
Bug 1013278
Opened 11 years ago
Closed 11 years ago
New Maker Party website isn't localized yet, causing some broken links on webmaker.org
Categories
(Webmaker Graveyard :: Maker Party, defect)
Webmaker Graveyard
Maker Party
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: panaggio, Assigned: panaggio)
References
()
Details
Attachments
(1 file)
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:32.0) Gecko/20100101 Firefox/32.0 (Beta/Release)
Build ID: 20140516030204
Steps to reproduce:
* Go to https://webmaker.org/pt-BR/about
* Click "Maker Party" link in the bottom
Actual results:
* Opened http://party.webmaker.org/pt-BR
* Got a "Cannot GET /pt-BR"
Expected results:
* Open http://party.webmaker.org/pt-BR properly
| Assignee | ||
Comment 1•11 years ago
|
||
I gave just one example. That happens in several other places. I could find some greping, but there could be more:
* ./public_angular/js/controllers.js: target: 'http://party.webmaker.org/' + CONFIG.lang + '/partners'
* ./views/teach.html: "url": "http://party.webmaker.org/{{localeInfo.lang}}"
* ./views/about.html: "url": "http://party.webmaker.org/{{localeInfo.lang}}"
* ./views/layout.html: "url": "http://party.webmaker.org/{{localeInfo.lang}}"
* ./views/privacy-makes.html: "url": "http://party.webmaker.org/{{localeInfo.lang}}"
* ./views/tools.html:"url": "http://party.webmaker.org/{{localeInfo.lang}}"
* ./locale/en_US/explore.json: "message": "The Webmaker community gets together at events around the world—at libraries, community centers, internet cafes, or in your own living room. <a href=\"https://events.webmaker.org/{{ lang }}/\">Find an event near you</a>, learn how to <a href=\"https://events.webmaker.org/{{ lang }}/#!/event-guides\">host your own</a>, or get involved with our global <a href=\"http://party.webmaker.org/{{ lang }}/\">Maker Party</a> (July to Sep 2014).",
Again, this is something I could fix myself, but I don't know if we should redirect to party.webmaker.org, or remove these broken links while party.webmaker.org isn't localized, ... If you want, I can fix, I just need a direction.
Comment 3•11 years ago
|
||
Makerparty is currently hard-coded to support only en-US, so we'll need to patch it to use a preference for supported languages: https://github.com/mozilla/makerparty/blob/master/app.js#L31
panaggio: Do you want to patch makerparty?
Updated•11 years ago
|
Assignee: nobody → panaggio
Status: NEW → ASSIGNED
| Assignee | ||
Comment 4•11 years ago
|
||
Working on it. First time I run makerparty locally, could take a while. But I'm on it till I finish it. Hope to have a patch and pull request it soon.
| Assignee | ||
Comment 5•11 years ago
|
||
Updated•11 years ago
|
Attachment #8425673 -
Flags: review?(jon)
Updated•11 years ago
|
Attachment #8425673 -
Flags: review?(jon) → review+
Updated•11 years ago
|
Component: General → Maker Party
QA Contact: brett
Comment 6•11 years ago
|
||
http://party.webmaker.org/pt-BR/ is now golden. Thanks :panaggio and :mavis!
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•