Skip to content

[GraphHopper] Add provider parameter #1251

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

bisleadity
Copy link

@bisleadity bisleadity commented Mar 21, 2025

Graphhopper geocoding now supports different providers. Hereby the option is given to use those providers.

As we do need to use one of those providers and there was no possibility given so far I added it.

Copy link
Member

@jbelien jbelien left a comment

Choose a reason for hiding this comment

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

This is a feature still in development.

The provider parameter is currently under development and can fall back to default at any time.
-- https://docs.graphhopper.com/openapi/geocoding/getgeocode#geocoding/getgeocode/t=request&in=query&path=provider

Graphhopper geocoding supports different providers. Hereby the option is
given to use those providers.
@bisleadity bisleadity force-pushed the feature/graphhopper-add-provider branch from 8fedd22 to 4d66689 Compare March 24, 2025 09:35
@bisleadity bisleadity requested a review from jbelien March 27, 2025 12:25
…aphHopper geocoding api

Graphhopper geocoding supports different providers. Hereby the option is given to use only available providers.

[GraphHopper] Add GraphHopperProviders enum for provider options of GraphHopper geocoding api

Graphhopper geocoding supports different providers. Hereby the option is given to use only available providers.
@bisleadity bisleadity force-pushed the feature/graphhopper-add-provider branch from 0fbc0e1 to 320b5d7 Compare March 28, 2025 09:16
…ns of GraphHopper geocoding api"

This reverts commit 320b5d7.
@jbelien
Copy link
Member

jbelien commented Apr 15, 2025

Could you maybe still add a real test?
Thanks!

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

Successfully merging this pull request may close these issues.

2 participants