Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(consumer): use newer LeaveGroup as appropriate #2544

Merged
merged 1 commit into from
Aug 3, 2023

Conversation

dnwe
Copy link
Collaborator

@dnwe dnwe commented Aug 2, 2023

Use the correct version and format of LeaveGroup as determined by the configured Version field.

Fixes #2486
Contributes-to: #2408

Use the correct version and format of LeaveGroup as determined by the
configured Version field.

Fixes #2486

Signed-off-by: Dominic Evans <dominic.evans@uk.ibm.com>
@dnwe dnwe added the fix label Aug 2, 2023
Base automatically changed from dnwe/is-valid-version to main August 3, 2023 09:46
Copy link
Collaborator

@hindessm hindessm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dnwe dnwe merged commit 017083e into main Aug 3, 2023
@dnwe dnwe deleted the dnwe/use-correct-leavegroup branch August 3, 2023 09:54
@dnwe dnwe added feat and removed fix labels Aug 7, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

LeaveGroupRequest still use the old version request
2 participants