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

Fix POST /ratings #189

Merged
merged 1 commit into from
Jan 19, 2025
Merged

Fix POST /ratings #189

merged 1 commit into from
Jan 19, 2025

Conversation

rgreinho
Copy link
Member

Fixes the POST /ratings endpoint by adding the missing score field.

The Open API Specifications and the bna client were updated accordingly.

A new integration test was added into the scenario tests.

Signed-off-by: Rémy Greinhofer [email protected]

Fixes the POST `/ratings` endpoint by adding the missing `score` field.

The Open API Specifications and the bna client were updated accordingly.

A new integration test was added into the scenario tests.

Signed-off-by: Rémy Greinhofer <[email protected]>
@rgreinho rgreinho self-assigned this Jan 19, 2025
@pull-request-size pull-request-size bot added the size/L Large (100-499 lines of changes) label Jan 19, 2025
@rgreinho rgreinho merged commit a780893 into main Jan 19, 2025
11 checks passed
@rgreinho rgreinho deleted the fix-post-rating branch January 19, 2025 23:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/L Large (100-499 lines of changes)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant