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

Improve OpenAPI definition #41

Merged
merged 8 commits into from
Jan 31, 2024
Merged

Conversation

dvacca-onfido
Copy link
Contributor

@dvacca-onfido dvacca-onfido commented Jan 23, 2024

  • Add missing verbs and models and remove standalone files when not necessary
  • Make use of enums and pattern validators when possible
  • Update README.md as well as YAML and JSON output files
  • Align the sorting of the different endpoints with public documentation

- Add missing verbs and models
- Use constant enum instead of discrimitator to chose appropriate models for reports
- Make use of enums and pattern validators when possible
- Update YAML and JSON output file
- Update README.md
@dvacca-onfido dvacca-onfido force-pushed the feature/improve_openapi_definition branch from a0bf380 to a14ff13 Compare January 23, 2024 11:02
- Improve regex definitions for pattern validation
- Use discriminator again and add enum for report types
- Rebuild YAML and JSON definitions with precise openapi generator version (7.2.0)
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
openapi.yaml Outdated Show resolved Hide resolved
@dvacca-onfido dvacca-onfido force-pushed the feature/improve_openapi_definition branch from f523c39 to f57d52a Compare January 26, 2024 15:36
@dvacca-onfido dvacca-onfido force-pushed the feature/improve_openapi_definition branch from 7985f55 to 401e073 Compare January 26, 2024 16:58
README.md Outdated Show resolved Hide resolved
generated/artifacts/openapi-yaml/.openapi-generator-ignore Outdated Show resolved Hide resolved
paths/checks.yaml Outdated Show resolved Hide resolved
schemas/extractions/extraction.yaml Outdated Show resolved Hide resolved
schemas/workflow_runs/workflow_run.yaml Show resolved Hide resolved
@dvacca-onfido dvacca-onfido force-pushed the feature/improve_openapi_definition branch from dc7283e to 2aceb17 Compare January 30, 2024 15:50
@dvacca-onfido dvacca-onfido force-pushed the feature/improve_openapi_definition branch from 5dfe625 to 6dbc01b Compare January 30, 2024 17:24
@dvacca-onfido dvacca-onfido force-pushed the feature/improve_openapi_definition branch from 6dbc01b to 1c4add9 Compare January 30, 2024 17:31
@dvacca-onfido dvacca-onfido merged commit 2f6f506 into master Jan 31, 2024
@HoneyryderChuck HoneyryderChuck deleted the feature/improve_openapi_definition branch January 31, 2024 13:40
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.

3 participants