Skip to content

[bug 989115] Implemented simple wiki landing page for configured locales#2145

Merged
rlr merged 1 commit intomozilla:masterfrom
rlr:faq-locales-989115
Dec 4, 2014
Merged

[bug 989115] Implemented simple wiki landing page for configured locales#2145
rlr merged 1 commit intomozilla:masterfrom
rlr:faq-locales-989115

Conversation

@rlr
Copy link
Contributor

@rlr rlr commented Oct 1, 2014

I implemented a decorator that requests certain configured locales to be redirected to a configured kb page. The settings still aren't set (waiting for the info) but the actual code is ready to go I think.

To test:

  • Run tests
  • Add 'es' to the 'SIMPLE_WIKI_LANGUAGES' setting. Go to '/es/products/firefox' and see it redirect you to '/es/kb/faq' which will be a 404 if you haven't created it.

r?

@rlr rlr force-pushed the faq-locales-989115 branch from ec1efa7 to 48892f7 Compare October 7, 2014 16:59
@rlr
Copy link
Contributor Author

rlr commented Oct 7, 2014

OK. This is ready with the right settings for shipping now. ☝️

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you can put this decorator on the class if you want to.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ooooh. cool. I'll try 👍

@mythmon
Copy link
Contributor

mythmon commented Oct 7, 2014

Travis 💣

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're checking for the wrong url here.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh right. I changed that in the last commit. #fail

@rlr
Copy link
Contributor Author

rlr commented Oct 7, 2014

❤️ travis

@mythmon
Copy link
Contributor

mythmon commented Oct 7, 2014

Besides the test failure, this seems fine to me.

I checked on the site, and none of these locales have any content on the FAQ yet. I'm not sure if we want to save to land this with the settings right now considering that.

@rlr
Copy link
Contributor Author

rlr commented Oct 7, 2014

fixes 👆

Yeah. I'm going to wait for green light from the L10n boss.

@mythmon
Copy link
Contributor

mythmon commented Oct 7, 2014

Cool. Code is r+.

@rlr rlr force-pushed the faq-locales-989115 branch from 45c3cd9 to 54fedec Compare November 3, 2014 13:28
@rlr
Copy link
Contributor Author

rlr commented Nov 3, 2014

rebased against master. still waiting for green light. le sigh

@mythmon
Copy link
Contributor

mythmon commented Nov 4, 2014

I see green. 🚢

@mythmon
Copy link
Contributor

mythmon commented Nov 4, 2014

Oh, oops. I thought you meant green from Travis, not from L10n boss. Nevermind.

@rlr rlr force-pushed the faq-locales-989115 branch 2 times, most recently from 470465d to c2db4a2 Compare December 4, 2014 19:16
@rlr
Copy link
Contributor Author

rlr commented Dec 4, 2014

rebased and limited to only estonian (et). I'll land as soon as Travis is cool.

@rlr rlr force-pushed the faq-locales-989115 branch from c2db4a2 to 66a43dd Compare December 4, 2014 19:31
@rlr rlr merged commit 66a43dd into mozilla:master Dec 4, 2014
@rlr
Copy link
Contributor Author

rlr commented Dec 4, 2014

66a43dd [bug 989115] Implemented simple wiki landing page for configured locales

@rlr rlr deleted the faq-locales-989115 branch December 5, 2014 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants