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

WINDUPRULE-826 SB2Quarkus Swagger and OpenAPI property settings #634

Merged
merged 2 commits into from
Mar 24, 2022

Conversation

PhilipCattanach
Copy link
Contributor

This PR contains rules for WINDUPRULE-826 and WINDUPRULE-827
They are both very similar in that they are SB2Quarkus rules pertaining to properties for Swagger and Open API respectively.
These rules originate from this tutorial https://github.com/RedHat-Middleware-Workshops/spring-to-quarkus-todo#migrate-data-source-properties

<perform>
<hint title="Replace Spring OpenAPI endpoint mapping" effort="1" category-id="mandatory">
<message>
Replace `springdoc.api-docs.path` with `quarkus.smallrye-openapi.path`
Copy link
Contributor

Choose a reason for hiding this comment

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

do we need to add something here to mention changing =/openapi to =/swagger-ui?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi @m-brophy ,
in both cases the value on the right side of the is consistent, after moving to quarkus.
It is just the name of the property that needs to be changed.

This is the solution view of the file https://github.com/RedHat-Middleware-Workshops/spring-to-quarkus-todo/blob/solution/src/main/resources/application.properties

This is the original SB version of the file
https://github.com/RedHat-Middleware-Workshops/spring-to-quarkus-todo/blob/main/src/main/resources/application.properties

@m-brophy m-brophy merged commit d0f7133 into windup:master Mar 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants