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

Update to MaxMind GeoIP2 .NET version 2.1.0 #6902

Merged
merged 1 commit into from Nov 11, 2014

Conversation

Mailaender
Copy link
Member

We were using a pre-release version before. I also updated the http://dev.maxmind.com/geoip/geoip2/geolite2/ database.

@Mailaender Mailaender changed the title Update to MaxMind GeoIP2 .NET version 2.10 Update to MaxMind GeoIP2 .NET version 2.1.0 Nov 7, 2014
@pchote
Copy link
Member

pchote commented Nov 10, 2014

I've confirmed that this at least compiles and runs, but there isn't any easy way to test the functionality locally. We will need to organize a dev game so that we can check a few location lookups in the lobby.

@Mailaender
Copy link
Member Author

Just go to the game server list. You may need to tweak your version string or enable a debug settings which ignores incompatible versions.

@pchote
Copy link
Member

pchote commented Nov 10, 2014

The incompatible checkbox replaces the country label with the mod + version string. Editing the mod.yaml to set release-20141029 works though. This appears to work, so 👍.

@obrakmann
Copy link
Contributor

👍

obrakmann added a commit that referenced this pull request Nov 11, 2014
Update to MaxMind GeoIP2 .NET version 2.1.0
@obrakmann obrakmann merged commit b37a90c into OpenRA:bleed Nov 11, 2014
@obrakmann
Copy link
Contributor

Changelog

@Mailaender Mailaender deleted the geoip-2.1.0 branch November 11, 2014 19:25
@Mailaender
Copy link
Member Author

https://github.com/OpenRA/OpenRA/blob/bleed/OpenRA.Game/Settings.cs#L74 was the developer setting I was talking about. It is not exposed to the GUI. I was asked to remove it for polishing reasons.

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

Successfully merging this pull request may close these issues.

None yet

3 participants