-
Notifications
You must be signed in to change notification settings - Fork 726
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
add clientName and heartBeat metadata for pubsub rabbitmq #3718
Conversation
Signed-off-by: zhangchao <[email protected]>
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.
quick review - thanks for this PR!
daprdocs/content/en/reference/components-reference/supported-pubsub/setup-rabbitmq.md
Outdated
Show resolved
Hide resolved
@Taction please look at the comment and resolve conflicts |
Signed-off-by: Hannah Hunter <[email protected]>
resolved conflicts |
…ubsub/setup-rabbitmq.md Co-authored-by: Hannah Hunter <[email protected]> Signed-off-by: Taction <[email protected]>
@hhunter-ms Thanks for your review and for resolving conflicts! |
@Taction - This PR is in the v1.11 docs, but the code is in v1.12. Can you move this PR to target the v1.12 docs? Thanks |
@msfussell Thanks for pointing out that. I opened another pr #3740 to target the v1.12 doc. |
Thank you for helping make the Dapr documentation better!
Please follow this checklist before submitting:
In addition, please fill out the following to help reviewers understand this pull request:
Description
Add missing content.
Issue reference
Please reference the issue this PR will close: #3710