Skip to content

Comments

[FS-1534] A subconversation leaver gets a remove proposal#3080

Merged
stefanwire merged 4 commits intomlsfrom
fs-1534/subconv-leaver-gets-rm-proposal
Feb 14, 2023
Merged

[FS-1534] A subconversation leaver gets a remove proposal#3080
stefanwire merged 4 commits intomlsfrom
fs-1534/subconv-leaver-gets-rm-proposal

Conversation

@mdimjasevic
Copy link
Contributor

This fixes a bug identified in https://wearezeta.atlassian.net/browse/FS-1534. A client leaving a subconversation would get a backend-created remove proposal, while it should not. The fix makes the application logic consistent with the MLS conversation leaving case where the leaving user gets no backend-created proposals.

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@mdimjasevic mdimjasevic marked this pull request as ready for review February 13, 2023 14:53
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Feb 13, 2023
@stefanwire stefanwire merged commit a7ba9f6 into mls Feb 14, 2023
@stefanwire stefanwire deleted the fs-1534/subconv-leaver-gets-rm-proposal branch February 14, 2023 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants