Skip to content

Include nodes, user, uid, timestamp, and version in GeoJSON properties #130

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Mar 2, 2021
Merged

Include nodes, user, uid, timestamp, and version in GeoJSON properties #130

merged 1 commit into from
Mar 2, 2021

Conversation

@russbiggs
Copy link
Contributor

@russbiggs russbiggs commented Dec 12, 2020

Currently verbosity parameter does not return additional properties in the GeoJSON output, so the following statements all return the same output:

api.get('way(696160849)', verbosity='tags') api.get('way(696160849)', verbosity='body') api.get('way(696160849)', verbosity='meta')

This PR puts the nodes, timestamp, user, uid, version into the GeoJSON properties object if they exist in the response data.

I've also updated the test example.json to match the changes. Happy to add an additional test for the full meta level if that is helpful.
@mvexel mvexel merged commit 32281d2 into mvexel:master Mar 2, 2021
@dericke dericke mentioned this pull request Mar 5, 2021
Closed
2 tasks
@dericke dericke mentioned this pull request Mar 15, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@russbiggs @mvexel