-
Notifications
You must be signed in to change notification settings - Fork 2
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
Data schema #4
Comments
@vgeorge this looks like a good start. Some questions: User SchemaHow do we deal with / store the relationship to the OSM auth in the schema? This looks good to me for a simple auth mechanism, not sure if we need to think about the schema in relation to logging in with OSM username / linking to osm user data. TracesWhile storing the Do we also need to store the PhotosAgain, we will need some geo-data attached to the photos to be able to do things like "find all photos in this bounding box" - perhaps just a Point field to hold a lat / lng? cc @geohacker The other question I have is around using Overall, this looks great - my main feedback is just ensuring that things will be filterable by bounding box / other queries we need. |
@vgeorge One note on username - I assume the account/database source for this needs to be configurable so that developers can implement the API for containerized versions of OSM. |
Added three changes to the schema:
@LanesGood I'm not sure if I understood correctly how this relates to the user schema, but the OAuth endpoint will be configurable, so it would be posible to use a custom OSM instance if it offers OAuth. |
One outstanding question from me. Do you see the need of labels for traces and/or images? |
@vgeorge sorry, one additional comment / request: can we have |
@batpad thanks for remembering this. I've added as a required number field. |
Regarding the trace schema, OSM has a https://www.openstreetmap.org/user/vgeorge/traces/1227616 It has also a |
A first version of the data schema is implemented. It's a little bit different from the description at the first comment, but in line with our current understading of features and use cases. Let's discuss any further changes in specific issues. |
Users
Traces
Photos
node/2932938
,way/10203932
,relation/102293
, etcThe text was updated successfully, but these errors were encountered: