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

Mixed to MLS transition #3334

Merged
merged 7 commits into from
Jun 7, 2023
Merged

Mixed to MLS transition #3334

merged 7 commits into from
Jun 7, 2023

Conversation

pcapriotti
Copy link
Contributor

@pcapriotti pcapriotti commented Jun 1, 2023

This PR implements the final transition step of MLS migration, i.e. from "mixed" protocol to MLS. In order to validate this transition, we first check if the MLS migration feature is enabled, then verify that all users in the conversation have MLS listed as a supported protocol.

https://wearezeta.atlassian.net/browse/FS-1582

TODO

  • Test removing extraneous clients

Checklist

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

@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Jun 1, 2023
@pcapriotti pcapriotti force-pushed the pcapriotti/mixed-to-mls2 branch from 7450140 to 0bcb446 Compare June 2, 2023 08:05
@pcapriotti pcapriotti changed the base branch from develop to mls June 2, 2023 08:58
@pcapriotti pcapriotti force-pushed the pcapriotti/mixed-to-mls2 branch 2 times, most recently from d39693d to 2a1dfb4 Compare June 5, 2023 08:18
@pcapriotti pcapriotti marked this pull request as ready for review June 5, 2023 13:06
@pcapriotti pcapriotti force-pushed the pcapriotti/mixed-to-mls2 branch from e0b22c1 to e90ab2b Compare June 7, 2023 11:42
@pcapriotti pcapriotti force-pushed the pcapriotti/mixed-to-mls2 branch from e90ab2b to 69af144 Compare June 7, 2023 12:29
@pcapriotti pcapriotti merged commit a17e265 into mls Jun 7, 2023
@pcapriotti pcapriotti deleted the pcapriotti/mixed-to-mls2 branch June 7, 2023 13:15
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