Skip to content

[WPB-6783] Support unblocking an MLS 1-to-1 conversation#3940

Merged
mdimjasevic merged 8 commits intodevelopfrom
wpb-6783/1-1-mls-conversation-when-unblocked
Mar 13, 2024
Merged

[WPB-6783] Support unblocking an MLS 1-to-1 conversation#3940
mdimjasevic merged 8 commits intodevelopfrom
wpb-6783/1-1-mls-conversation-when-unblocked

Conversation

@mdimjasevic
Copy link
Contributor

@mdimjasevic mdimjasevic commented Mar 11, 2024

The PR adds support for unblocking a user in an MLS 1-to-1 conversation.

Tracked by https://wearezeta.atlassian.net/browse/WPB-6783.

Checklist

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

@mdimjasevic mdimjasevic force-pushed the wpb-6783/1-1-mls-conversation-when-unblocked branch from 56c3136 to 442b067 Compare March 11, 2024 14:22
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Mar 11, 2024
@mdimjasevic mdimjasevic force-pushed the wpb-6783/1-1-mls-conversation-when-unblocked branch 3 times, most recently from 8fd5635 to e6734a4 Compare March 12, 2024 11:09
@mdimjasevic mdimjasevic marked this pull request as ready for review March 12, 2024 11:13
@mdimjasevic mdimjasevic force-pushed the wpb-6783/1-1-mls-conversation-when-unblocked branch 2 times, most recently from 340ddaf to 27ca5de Compare March 12, 2024 12:45
Copy link
Contributor

@stefanwire stefanwire left a comment

Choose a reason for hiding this comment

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

Feel free to ignore the nits.

Marko Dimjašević added 8 commits March 13, 2024 08:51
This might need some more work, but it's pretty much there. The
potentially missing part is what should 'bob1' get on its web socket.
It is not clear yet this is everything that needs to be done. However,
it seems like the backend cannot just readd the MLS clients of the user
that unblocked the other user, as it has no leaf indices available for a
remotely hosted one-to-one conversation.
Moved the logic from `Brig.IO.Intra` to the `GalleyProvider` effect
@mdimjasevic mdimjasevic force-pushed the wpb-6783/1-1-mls-conversation-when-unblocked branch from 7275f59 to 9e016c8 Compare March 13, 2024 07:54
@mdimjasevic mdimjasevic merged commit bf5b6ff into develop Mar 13, 2024
@mdimjasevic mdimjasevic deleted the wpb-6783/1-1-mls-conversation-when-unblocked branch March 13, 2024 09:25
@echoes-hq echoes-hq bot added the echoes: unplanned Any work item that isn’t part of the product or technical roadmap. label Jul 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

echoes: unplanned Any work item that isn’t part of the product or technical roadmap. 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