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

update options to return datasources labels #5576

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

maxgrossman
Copy link
Contributor

updates options response.

I didn't touch the config.json since I know we have the overrides mechanism so figured that would be how we'd update the data source objects.

@maxgrossman
Copy link
Contributor Author

I mischaracterized this feature in the above comment.

I convinced myself we had a config.local.json that we could stick these optional labels in.

Anyhow, how about these.

API

Retrieves data from a server that implements v0.6 of the OSM API. This datasource support larger area downloads.
(Because it uses chunked requests via ParallelBoundedAPIReader?)

APIDB

Retrieves data through a direct connection to an OSM database.

@brianhatchl
Copy link
Contributor

Thinking this change would require changes to export server clients too.

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.

2 participants