-
-
Notifications
You must be signed in to change notification settings - Fork 6
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
Croatia not coloured green in overview map #389
Comments
I believe the tiles that make up the map with highlighted regions is heavily cached. It might be that those need to get flushed? |
Cache seems to be very heavy indeed... 😄 |
Yes – I think it's an indefinite cache and I'm not sure how to clear it. @ingalls any ideas here? |
I've tried to debug from the client side... By doing partial blocking in firefox developer console, it seems that the stale data is coming from https://batch.openaddresses.io/api/map/0/0/0.mvt (and other closer zooms too). Unfortunately the client-facing web server does not provide Which in turn would seem to indicate that perhaps those global HTTP header details
The fact it colors normally when specifying county directly (e.g. via https://batch.openaddresses.io/location/862) seems not related to the issue, as that seems to use MultiPolygon GeoJSON to color green the specified country (e.g. via https://batch.openaddresses.io/api/map/862) regardless if it has any data in openaddresses or not. TL;DR: someone with access to the server would probably need to check why those |
Describe the bug
Croatia is not marked as covered on the overview map, even though it was added in openaddresses/openaddresses#7172 and is being covered in https://batch.openaddresses.io/location/862
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Croatia should be coloured green on the map
Screenshots
https://batch.openaddresses.io/data
https://batch.openaddresses.io/location/862
Desktop (please complete the following information):
Additional context
There was a similar problem with Kosovo openaddresses/machine#700 but that was a bit more politically problematic area.
The text was updated successfully, but these errors were encountered: