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

refactor: introduce record to represent create participant response #498

Merged
merged 1 commit into from
Dec 5, 2024

Conversation

ndr-brt
Copy link
Member

@ndr-brt ndr-brt commented Dec 5, 2024

What this PR changes/adds

Introduce a CreateParticipantContextResponse record that replaces the generic Map used.
Add it to the openapi documentation (as currently it shows an empty body)

Why it does that

Enhance domain model, fix documentation

Further notes

List other areas of code that have changed but are not necessarily linked to the main feature. This could be method
signature changes, package declarations, bugs that were encountered and were fixed inline, etc.

Linked Issue(s)

Closes #497

Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.

@ndr-brt ndr-brt added documentation Improvements or additions to documentation refactoring Changing directories, modules, structures, etc. without functional changes api labels Dec 5, 2024
Copy link
Member

@paullatzelsperger paullatzelsperger left a comment

Choose a reason for hiding this comment

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

LGTM

@ndr-brt ndr-brt force-pushed the 497-set-response-type branch 2 times, most recently from 5c2b572 to e11289d Compare December 5, 2024 13:06
@ndr-brt ndr-brt force-pushed the 497-set-response-type branch from e11289d to 8a3c60e Compare December 5, 2024 13:13
@ndr-brt ndr-brt merged commit a66ddd2 into main Dec 5, 2024
14 checks passed
@ndr-brt ndr-brt deleted the 497-set-response-type branch December 5, 2024 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api documentation Improvements or additions to documentation refactoring Changing directories, modules, structures, etc. without functional changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

docs(api): response type on POST /v1alpha/participants is not set correctly
3 participants