Skip to content

Lcra/update required data organization openapi prod 14189 #851

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

Merged

Conversation

Leopold-Cramer
Copy link
Contributor

No description provided.

… conventions

- Changed collection GET endpoints to use 'list' prefix
- Changed single resource GET endpoints to use 'get' prefix
- Changed POST endpoints to use 'create' prefix
- Changed DELETE endpoints to use 'delete' prefix
- Updated security-related operation names for consistency
- Maintained existing operation IDs that already followed conventions
- added required to key values in openapi.yaml for organization
- removed warnings due to useless non-null assertions
- removed a few easy to fix warnings
@Leopold-Cramer Leopold-Cramer force-pushed the LCRA/update_required_data_organization_openapi_PROD-14189 branch from 9558d55 to e48195a Compare January 24, 2025 09:52
Copy link
Member

@jreynard-code jreynard-code left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM ;)

@Leopold-Cramer Leopold-Cramer merged commit f65f199 into main Jan 24, 2025
33 checks passed
@Leopold-Cramer Leopold-Cramer deleted the LCRA/update_required_data_organization_openapi_PROD-14189 branch January 24, 2025 12:42
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.

2 participants