Skip to content
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.

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

Fixed #30447 - Added 'is_in_european_union' in GeoIP2.city() dict. #11327

Merged
merged 1 commit into from May 7, 2019

Conversation

c23gooey
Copy link
Contributor

@c23gooey c23gooey commented May 6, 2019

Maxmind data supplies european union information as part of both the free and paid database sets.
This isn't accessible from the GeoIP2 city dict, despite it being available.

@c23gooey c23gooey changed the title #30447 - Added 'is_in_european_union' to GeoIP2.city() dict Fixed #30447 - Added 'is_in_european_union' to GeoIP2.city() dict May 6, 2019
@felixxm felixxm self-assigned this May 6, 2019
@felixxm felixxm changed the title Fixed #30447 - Added 'is_in_european_union' to GeoIP2.city() dict Fixed #30447 - Added 'is_in_european_union' in GeoIP2.city() dict. May 7, 2019
@felixxm felixxm merged commit 705ef6c into django:master May 7, 2019
@browniebroke
Copy link
Sponsor Contributor

I was trying to update to Django 3.0 and this new field caused an AttributeError. In case someone else have the same issue, the fix is to update GeoIP2 to 2.7.0+ (yes it's more than 2 years old 😄).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants