-
Notifications
You must be signed in to change notification settings - Fork 77
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): added annotation for google pubsub message binding #677
Conversation
✅ Deploy Preview for springwolf-ui canceled.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice addition @SheheryarAamir
...n/java/io/github/springwolf/examples/cloudstream/configuration/CloudstreamConfiguration.java
Show resolved
Hide resolved
...s/springwolf-googlepubsub-binding/src/test/java/GooglePubSubMessageBindingProcessorTest.java
Show resolved
Hide resolved
...b/springwolf/bindings/googlepubsub/annotations/channels/GooglePubSubAsyncChannelBinding.java
Outdated
Show resolved
Hide resolved
…ding from auto-configuration
...ngwolf/bindings/googlepubsub/annotations/channels/GooglePubsubAsyncMessageStoragePolicy.java
Outdated
Show resolved
Hide resolved
...ub/springwolf/bindings/googlepubsub/annotations/messages/GooglePubSubAsyncMessageSchema.java
Outdated
Show resolved
Hide resolved
Nice addition and improvement of Google PubSub Message bindings. We commented on two things. |
…t and retention annotation
Hi @SheheryarAamir, In the end I arrived at SheheryarAamir#1 EDIT: I was able to cherry-pick and push it to this branch. Hopefully, the tests are green. I will merge it then. |
Added annotation for google pubsub message binding