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

feat: Added support for Message description #211

Merged

Conversation

ctasada
Copy link
Collaborator

@ctasada ctasada commented Jun 6, 2023

While working at #189 we found out that the provided Message description was, in fact, the one associated to the Operation. As a result we decided to fully remove the description until we could fix it.

This PR adds support for Message description.

Any Message object, annotated with @Schema will use the annotation description value, as the Message description.

The https://www.springwolf.dev/docs/configuration/documenting-schemas documentation is already stating this scenario, so I don't think we need further documentation updates.

@ctasada ctasada force-pushed the ctasada/add-message-description branch from 630863f to 52ace14 Compare June 6, 2023 16:49
While working at springwolf#189 we found out that the provided Message description was, in fact, the one associated to the Operation. As a result we decided to fully remove the description until we could fix it.

This PR adds support for Message description.

Any `Message` object, annotated with `@Schema` will use the annotation `description` value, as the Message description.

The https://www.springwolf.dev/docs/configuration/documenting-schemas documentation is already stating this scenario, so I don't think we need further documentation updates.
@ctasada ctasada force-pushed the ctasada/add-message-description branch from 52ace14 to 8992c56 Compare June 7, 2023 06:33
@timonback
Copy link
Member

Looks good, thanks for fixing this!

@timonback timonback merged commit 4b32eb5 into springwolf:master Jun 7, 2023
@ctasada ctasada deleted the ctasada/add-message-description branch December 13, 2023 17:05
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