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

fix: Improved error message #436

Merged

Conversation

ctasada
Copy link
Collaborator

@ctasada ctasada commented Nov 6, 2023

When a method defining @AsyncGenericOperationBinding has more than 1 argument and misses the @Payload annotation, the provided exception message was confusing.

Instead of Multi-parameter KafkaListener methods must have one parameter annotated with @Payload but none was found we now provide a more generic Multi-parameter AsyncListener methods must have one parameter annotated with @Payload but none was found

Copy link

netlify bot commented Nov 6, 2023

Deploy Preview for springwolf-ui canceled.

Name Link
🔨 Latest commit 61a9df2
🔍 Latest deploy log https://app.netlify.com/sites/springwolf-ui/deploys/654a3cb399e263000873ca92

When a method defining `@AsyncGenericOperationBinding` has more than 1 argument and misses the `@Payload` annotation, the provided exception message was confusing.

Instead of `Multi-parameter KafkaListener methods must have one parameter annotated with @payload but none was found` we now provide a more generic `Multi-parameter AsyncListener methods must have one parameter annotated with @payload but none was found`
@ctasada ctasada force-pushed the ctasada/fix-listener-error-message branch from a0df172 to 61a9df2 Compare November 7, 2023 13:33
@timonback timonback merged commit 98046d0 into springwolf:master Nov 10, 2023
13 checks passed
@ctasada ctasada deleted the ctasada/fix-listener-error-message branch March 29, 2024 11:49
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.

None yet

2 participants