Skip to content

Add continent code(s) assignment #226

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Bost wants to merge 7 commits into from
Closed

Add continent code(s) assignment #226

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
f6640cb
assert_equal: formatted json-strings make debugging easier
BostX Mar 26, 2020
0ca8f68
Readable json test files
BostX Mar 27, 2020
937648e
Add continent code(s) assignment
BostX Mar 26, 2020
336c2e4
Document continent_codes_list in the README.md
BostX Mar 26, 2020
a6094ff
Fix formatting and import order
BostX Mar 27, 2020
09e02b5
Fix continent codes form a set. Not a list
BostX Mar 28, 2020
ee02187
Egypt is located in Africa and also in Asia
BostX Mar 28, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Egypt is located in Africa and also in Asia
  • Loading branch information
@BostX
BostX committed Mar 29, 2020
commit ee02187de95cb428696128f48d00eca564f26c91
5 changes: 4 additions & 1 deletion app/utils/continents.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -256,7 +256,7 @@
"UG": {"AFR"},
"UA": {"EUR"},
"MK": {"EUR"},
"EG": {"AFR"},
"EG": {"ASI" "AFR"},
"GB": {"EUR"},
"GG": {"EUR"},
"JE": {"EUR"},
Expand Down Expand Up @@ -302,6 +302,9 @@ def continent_codes_set(s):
TR - Turkey
KZ - Kazakhstan
AZ - Azerbaijan

ASI - Asia and AFR - Africa
EG - Egypt
"""
codes_set = country_code__continent_codes_set.get(s, default_continent_codes_set)
if codes_set == default_continent_codes_set:
Expand Down