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

Correct nominatim URL format #268

Merged
merged 1 commit into from
Aug 17, 2023

Conversation

danielmwilson
Copy link
Contributor

Remove trailing '/' from nominatim URL as this URL format is no longer supported:
/search/?q=query URL format no longer supported (should be /search?q=query) #3134
osm-search/Nominatim#3134

Remove trailing '/' from nominatim URL as this URL format is no longer supported:
 /search/?q=query URL format no longer supported (should be /search?q=query) #3134 
osm-search/Nominatim#3134
Copy link

@corey-wynn corey-wynn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we merge this?

@100jecki
Copy link

Sure! 👍

Copy link

@100jecki 100jecki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's ok. Change forced by OSM.

Copy link

@xy4u xy4u left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed ok. Change forced by OSM.

@jonataswalker jonataswalker merged commit 205e8d9 into Dominique92:master Aug 17, 2023
@jonataswalker
Copy link
Collaborator

Published 4.2.0

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 this pull request may close these issues.

5 participants