Skip to content

Refactor to use modern React #27

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 13 commits into
base: master
Choose a base branch
from

Conversation

tolicodes
Copy link

No description provided.

@gsingh-location
Copy link

Hi @tolicodes,

When are you guys planning to submit this release?
Also, Will this geocoder contain localGeocoder option?

@tolicodes
Copy link
Author

@gsingh-location I am not on the Mapbox Team. Actually not entirely sure they even saw my PR. But you can feel free to clone my repo.

What does the localGeocoder option do?

@treighton
Copy link

this is great, i wish i would have seen this 2 hours ago...

@tolicodes
Copy link
Author

Haha thanks

@tmcw any chance you could take a look at this? Would be great if the community could use this. I've got Storybook examples and all :)

@tmcw
Copy link
Contributor

tmcw commented Nov 16, 2018

I no longer work at Mapbox, thus no longer have review rights or responsibilities.

@josephbisaillon
Copy link

bummer this probably won't make it in. Thanks for the code though @tolicodes this is awesome.

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