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

Change the OpenAPI config mapping to use Config Interceptors #18936

Merged
merged 1 commit into from
Jul 23, 2021
Merged

Change the OpenAPI config mapping to use Config Interceptors #18936

merged 1 commit into from
Jul 23, 2021

Conversation

phillip-kruger
Copy link
Member

Fix #18900

As discussed with @radcortez .

Signed-off-by:Phillip Kruger [email protected]

@phillip-kruger
Copy link
Member Author

Thanks @geoand - I am going to create another PR for the GraphQL mapping, to move that to build time (like I have done here) rather than runtime.

@phillip-kruger phillip-kruger added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Jul 22, 2021
@geoand
Copy link
Contributor

geoand commented Jul 22, 2021

Thanks @geoand - I am going to create another PR for the GraphQL mapping, to move that to build time (like I have done here) rather than runtime.

Great!

@phillip-kruger phillip-kruger merged commit 20a87d6 into quarkusio:main Jul 23, 2021
@quarkus-bot quarkus-bot bot added this to the 2.2 - main milestone Jul 23, 2021
@quarkus-bot quarkus-bot bot removed the triage/waiting-for-ci Ready to merge when CI successfully finishes label Jul 23, 2021
@phillip-kruger phillip-kruger deleted the openapi-config-source-interceptor branch July 23, 2021 05:27
@gsmet gsmet modified the milestones: 2.2 - main, 2.1.1.Final Aug 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

quarkus-smallrye-openapi property quarkus.smallrye-openapi.operation-id-strategy doesn't work
4 participants