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

Geometry for event_location within the CDS Event API is not defined #129

Open
michael-danko-passport opened this issue Sep 15, 2023 · 2 comments
Labels
minor update A change that is minor and should require little discussion, or is a maintenance/readme/typo update.
Milestone

Comments

@michael-danko-passport
Copy link
Collaborator

Here: https://github.com/openmobilityfoundation/curb-data-specification/tree/main/events#curb-event

We list out GeoJSON but do not specify which type of geometry this is. The GeoJSON spec standard can be found here but we need to know what to document into the OpenAPI. https://datatracker.ietf.org/doc/html/rfc7946

I'm assuming this is a GeoJSON point but want to flag this that the documentation I first linked to needs to be updated.

@michael-danko-passport michael-danko-passport changed the title event_location within the CDS Event API is not defined Geometry for event_location within the CDS Event API is not defined Sep 15, 2023
@schnuerle
Copy link
Member

We give an example of a point here, and define an example of a polygon here, but yes in CDS events event_location we should say "GeoJSON Point" in the spec.

@schnuerle schnuerle transferred this issue from openmobilityfoundation/cds-openapi Feb 6, 2024
@schnuerle schnuerle added this to the 1.0.1 milestone Feb 6, 2024
@schnuerle schnuerle added the minor update A change that is minor and should require little discussion, or is a maintenance/readme/typo update. label Feb 6, 2024
@schnuerle
Copy link
Member

schnuerle commented Feb 6, 2024

A Point example is now given here and can be linked to in 1.1 when Objects are done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor update A change that is minor and should require little discussion, or is a maintenance/readme/typo update.
Projects
None yet
Development

No branches or pull requests

2 participants