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

version flag for MapboxGeocoder #485

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

VladislavMedved
Copy link
Contributor

@VladislavMedved VladislavMedved commented Jan 17, 2023

Description

PR adds version="v5"|"v6" flag to the options of MapboxGeocoder class that allows using v6 endpoint instead of default v5 when geocoding.

Gl-geocoder in v6 mode can initiate forward and reverse geocoding calls

Copy link
Contributor

@mpothier mpothier left a comment

Choose a reason for hiding this comment

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

@VladislavMedved Great work, this is really close - just a few small changes

lib/utils.js Outdated Show resolved Hide resolved
lib/utils.js Outdated Show resolved Hide resolved
lib/utils.js Outdated Show resolved Hide resolved
lib/index.js Outdated Show resolved Hide resolved
lib/index.js Outdated Show resolved Hide resolved
lib/index.js Outdated Show resolved Hide resolved
debug/index.js Outdated Show resolved Hide resolved
debug/index.js Outdated Show resolved Hide resolved
lib/utils.js Outdated Show resolved Hide resolved
lib/index.js Outdated Show resolved Hide resolved
lib/index.js Outdated Show resolved Hide resolved
debug/index.js Outdated Show resolved Hide resolved
@mapbox-danny
Copy link
Contributor

Be sure to remove the yarn lock

pre-validate reverse coordinates to avoid v6 4xx response

fix limit=1 on reverse geocodes when single type set

enable multiple reverse results when no types or limit specified

fix promise resolution

revert package-lock file

fix potential problem with address strings for v5 and v6
@VladislavMedved
Copy link
Contributor Author

VladislavMedved commented May 18, 2023

  • Update the version number in package.json and package-lock.json
  • Run npm run docs and commit changes to API.md
  • Update CHANGELOG.md with changes under main heading before merging

@VladislavMedved VladislavMedved changed the title useGeocoderV6 flag for MapboxGeocoder version flag for MapboxGeocoder May 18, 2023
@MateuszKukiela
Copy link

Is anyone still working on this? It would be really cool to see API v6 in the geocoder

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.

4 participants