Language switcher doesn't list locale names

VERIFIED FIXED

Status

Websites
markup.mozilla.org
P1
major
VERIFIED FIXED
7 years ago
7 years ago

People

(Reporter: stas, Unassigned)

Tracking

({l12y})

Firefox 4

Details

(Reporter)

Description

7 years ago
I think this had to do with the attachment 530471 [details] [diff] [review] from bug 652912.

::: ffdemo/settings.py
@@ +61,5 @@
>          from product_details import product_details
> +        return dict([(lang.lower(), product_details.languages[lang]['native'])
> +                     for lang in KNOWN_LANGUAGES])
> +
> +LANGUAGES = lazy(LazyLangs, dict)()

Austin, any specific reason why you changed the tuple to a dict?  I think this broke the lang switcher in the footer.

Comment 1

7 years ago
(In reply to comment #0)
This data structure is required by our middleware.
I'll look at updating the lang drop down.

Fixed in 
https://github.com/mozilla/markup/commit/9b6900b128999e759dd407cd82711d4525db8234
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
(Reporter)

Comment 2

7 years ago
Verified.  Thanks Austin.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.