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

Draft: make sure that @GraphQLClientApi attributes can be overriden by quarkus config #44441

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jmini
Copy link
Contributor

@jmini jmini commented Nov 12, 2024

Fixes: #44334

Add test case for #44334

Requires the code change of smallrye/smallrye-graphql#2223 so we need an update of the SmallRye GraphQL client to 2.12.0 (or 2.11.1) before being able to merge this PR.

Tested by @jmartisk locally as indicated in #44334 (comment)

Copy link

quarkus-bot bot commented Nov 12, 2024

Thanks for your pull request!

Your pull request does not follow our editorial rules. Could you have a look?

  • title should not contain an issue number (use Fix #1234 in the description instead)

This message is automatically generated by a bot.

@jmini jmini changed the title Draft: Add test case for #44334 Draft: make sure that @GraphQLClientApi attributes can be overriden by quarkus config Nov 12, 2024
@jmartisk jmartisk self-assigned this Nov 12, 2024
@jmartisk jmartisk self-requested a review November 12, 2024 12:42
@jmini jmini marked this pull request as draft November 12, 2024 12:56
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.

GraphQL client: "endpoint" parameter in "@GraphQLClientApi" has precedence over "application.properties"
2 participants