Closed
Bug 733955
Opened 14 years ago
Closed 13 years ago
[bedrock] Don't hardcode country list on collusion page
Categories
(www.mozilla.org :: General, defect)
www.mozilla.org
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: wenzel, Unassigned)
References
Details
The collusion page has a huge list of countries inline in the template:
<https://github.com/mozilla/bedrock/blob/28b3f3377de31a3566c7e134ba9718bdf11dcbde/apps/collusion/templates/collusion/collusion.html#L104>
Instead, we should feed this list straight from product-details:
>>> from product_details import product_details
>>> product_details.get_regions('en-US')
{u'ad': u'Andorra',
u'ae': u'U.A.E.',
u'af': u'Afghanistan',
u'ag': u'Antigua and Barbuda',
u'ai': u'Anguilla',
u'al': u'Albania',
u'am': u'Armenia',
...
}
(note this is a dict so we need to sort it. Also, cf. bug 733417).
| Assignee | ||
Updated•14 years ago
|
Component: www.mozilla.org/firefox → www.mozilla.org
| Assignee | ||
Updated•13 years ago
|
Component: www.mozilla.org → General
Product: Websites → www.mozilla.org
Comment 1•13 years ago
|
||
This has been fixed in https://github.com/mozilla/bedrock/pull/415
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•