Closed
Bug 1202653
Opened 9 years ago
Closed 4 years ago
Update geo system to use GENC list of countries and correct terminology
Categories
(Mozilla Reps Graveyard :: reps.mozilla.org, enhancement, P2)
Mozilla Reps Graveyard
reps.mozilla.org
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: gerv, Assigned: emin)
Details
(Whiteboard: [profiles][events])
The reps portal has a Location field which is split into three parts. One of those parts is called "Country" in the UI. As per the recent decision from Legal and explained in mozilla.governance, this needs to be changed as follows: a) The top-level categorization needs to be labelled "Country or Region". Mozilla has no opinion on the status or otherwise of any area as a "Country". b) The list of countries available to choose needs to be only those in the GENC list, with the names given in the GENC list. See https://github.com/gerv/genc2json for the data. Thanks, Gerv
Updated•7 years ago
|
Priority: -- → P2
Comment 1•6 years ago
|
||
I am working on this ticket, https://github.com/mozilla/remo/pull/1446 I made the first part on changing the label. For the second one I need feedback, actually the repo is not updated since few years and the code is using a json in `remo/lib/product_details_json/regions/en-US.json` (I guess that is using the US version). Documenting about this set seems that GENC is a modified version of ISO 3166-1 to use on US stuff (https://www.dni.gov/index.php/who-we-are/organizations/ic-cio/ic-cio-related-menus/ic-cio-related-links/ic-technical-specifications/geopolitical-entities-names-and-codes). For Django already exist a module that implemented a picker for countries on ISO 3166-1, so I think that we can use that and remove the local dependence of this json (https://github.com/SmileyChris/django-countries). That module implements also Flag so I think that we can use them also to improve the UI.
Assignee: nobody → mte90net
Comment 2•6 years ago
|
||
Without knowing that part of the code, I'm not sure why we have those files checked into our source control: https://github.com/mozilla/remo/tree/master/lib/product_details_json As far a I remember, those get downloaded from somewhere anyway if you build the Docker container. The same *should* be happening for production. Daniele, can you verify that the list on the Portal (production) matches the list in https://github.com/mozilla/product-details-json/tree/master/product-details and that it is actually true that it downloads the latest versions? If so, I guess we could remove https://github.com/mozilla/remo/tree/master/lib/product_details_json completely?
Flags: needinfo?(mte90net)
Comment 3•6 years ago
|
||
Checked on the remo github we have an old version infact, we don't have Kosovo in the country list but is available in the product-details-json repo. So I think that docker is not downloading them (https://github.com/mozilla/remo/commit/59a508653c42f85f89f8c66c11d10b0c0518ad77). I think that we need to give an high priority on updated this list, maybe we have reps that feel excluded because their country is not available. We need to evaluate if we want drop the idea of GENC/ISO list, I think hhat mozilla is using this list so is better to use that. The first things to do can be update that json and later improve the docker script to download automatically but countries seems that are not updated since 2 years so maybe is not so high the priority for this other step.
Flags: needinfo?(mte90net)
Comment 4•6 years ago
|
||
Also the updates of countries on that repo is made as I can see monthly but there are no changes so I can confirm that to fix now we can copy that repo in our.
Comment 5•6 years ago
|
||
(In reply to Daniele "Mte90" Scasciafratte from comment #4) > Also the updates of countries on that repo is made as I can see monthly but > there are no changes so I can confirm that to fix now we can copy that repo > in our. Can you create a new bugzilla ticket to figure out why it doesn't work automatically? Then for now I agree we can just update the list in our repo.
Updated•6 years ago
|
Status: NEW → ASSIGNED
Whiteboard: [profiles][events]
Updated•5 years ago
|
Type: task → enhancement
Updated•5 years ago
|
Assignee: mte90net → emin
Comment 6•4 years ago
|
||
As we've moved from reps.mozilla.org to the Community Portal, I'm closing this bug. Filter out these emails by searching for "REPS_PORTAL_BUG_CLOSURE"
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → WONTFIX
Updated•4 years ago
|
Product: Mozilla Reps → Mozilla Reps Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•