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

Photon #293

Closed
Wire64 opened this issue Oct 26, 2020 · 0 comments · Fixed by #296
Closed

Photon #293

Wire64 opened this issue Oct 26, 2020 · 0 comments · Fixed by #296

Comments

@Wire64
Copy link
Contributor

Wire64 commented Oct 26, 2020

Hi,

L.Control.Geocoder.photon().reverse(lon=10&lat=52) used to work (until this week).
It requests: https://photon.komoot.io/api?lon=10&lat=52
result: 400 (Bad Request)

But according to photon it now needs to request:
https://photon.komoot.io/reverse?lon=10&lat=52

Is this error because photon changed the api?

photon.komoot.de also changed to photon.komoot.io

--
update:

It failed because the redirect. I fixed it locally by changing:
src/geocoders/photon.js

6,7c6,7
< serviceUrl: 'https://photon.komoot.de/api/',
< reverseUrl: 'https://photon.komoot.de/reverse/',

serviceUrl: 'https://photon.komoot.io/api/',
reverseUrl: 'https://photon.komoot.io/reverse/',

Then it worked like expected.

Thanks,
Wire64

Wire64 added a commit to Wire64/leaflet-control-geocoder that referenced this issue Oct 26, 2020
Until October 2020 the API was available under photon.komoot.de. New url is photon.komoot.io.
Fixes reverse issue after change
see: perliedman#293
simon04 pushed a commit that referenced this issue Oct 28, 2020
Until October 2020 the API was available under photon.komoot.de. New url is photon.komoot.io.
Fixes reverse issue after change
see: #293
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant