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

Configurable directions/geocode hosts #79

Open
wants to merge 1 commit into
base: mb-pages
Choose a base branch
from

Conversation

freenerd
Copy link

@freenerd freenerd commented Jul 2, 2015

For testing internal staging services, we want to have configurable hosts.

/cc @jfirebaugh @lbud

@jfirebaugh
Copy link
Contributor

Some overlap with #50 here, though L.mapbox.config.HTTPS_URL doesn't permit different tile / geocoding / directions hosts.

@freenerd
Copy link
Author

freenerd commented Jul 2, 2015

@jfirebaugh i'm pretty sure we'll want different hosts. we might even add different access tokens then as well, but not sure that is needed ...

@lbud
Copy link
Contributor

lbud commented Jul 3, 2015

we'll want different hosts

Agree; staging directions and staging geocoding are very independent use cases.

@freenerd freenerd force-pushed the configurable-hosts branch from d9e7751 to a005c32 Compare July 4, 2015 20:12
@freenerd
Copy link
Author

freenerd commented Jul 4, 2015

Added tests and documentation

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.

3 participants