lacuna-tech / mds-core

Repo for LADOT MDS implementation for contribution to the Open Mobility Foundation
Apache License 2.0
13 stars 9 forks source link

Swagger-specifications for geography #976

Open mrsimpson opened 2 years ago

mrsimpson commented 2 years ago

📚 Purpose

This PR includes swagger definitions for the geography API. Actually, this could/should be part of the OMF spec repo, but as the version in mds core is not the most current one and e. g. there cannot be a commit to the 1.0-branch of the spec in the past, I'll focus on adding it here

👌 Includes (WIP state):

/geographies

Remark: In order to make this PR merge-able earlier, I removed the policy-related API from this PR. I'll create another one for that purpose

changeset-bot[bot] commented 2 years ago

⚠️ No Changeset found

Latest commit: 71db375745f71afb21ba9eff4917f6cc84c8a6d2

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

mrsimpson commented 2 years ago

@avatarneil wdyt? Does that help (without using it for schema generation or validation within the api-service)

avatarneil commented 2 years ago

@mrsimpson this looks great! I think even though this isn't using any of the schema generation or anything, it's still useful from a documentation perspective.

mrsimpson commented 2 years ago

@avatarneil I reduced this PR to /geography. I got sidetracked by other stuff and will not be able to complete the policy part in the next days. Therefore, feel free to merge this not holistic but imho consistent PR