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 geocoder.ex to include Territories #3868

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mattew124
Copy link

The address returned by the following link returns a territory in place of state so I have added state aliases to include this information:

https://nominatim.openstreetmap.org/reverse?format=xml&lat=-35.1604&lon=149.1049&zoom=19&addressdetails=1

The address returned by the following link returns Territory instead of state so I have added state aliases to include this information:

https://nominatim.openstreetmap.org/reverse?format=xml&lat=-35.1604&lon=149.1049&zoom=19&addressdetails=1
Copy link

netlify bot commented Apr 25, 2024

Deploy Preview for teslamate ready!

Name Link
🔨 Latest commit 84ce69e
🔍 Latest deploy log https://app.netlify.com/sites/teslamate/deploys/662a05d2ea310400081a50b0
😎 Deploy Preview https://deploy-preview-3868--teslamate.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@mattew124
Copy link
Author

Once this change is applied is it possible to update existing data?

@JakobLichterfeld JakobLichterfeld added the area:openstreetmap Related to OpenStreetMap label Apr 27, 2024
@JakobLichterfeld
Copy link
Collaborator

Thanks for your suggestion!

Once this change is applied is it possible to update existing data?

You can code a migration step which can add it for old data, not sure about the performance in this case tbh.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:openstreetmap Related to OpenStreetMap
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants