-
Notifications
You must be signed in to change notification settings - Fork 290
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
feat: user joins mls group with guest link #14832
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #14832 +/- ##
==========================================
+ Coverage 42.78% 42.89% +0.10%
==========================================
Files 622 622
Lines 21257 21280 +23
Branches 4880 4886 +6
==========================================
+ Hits 9095 9127 +32
+ Misses 11003 10993 -10
- Partials 1159 1160 +1 |
68f4026
to
0fb2146
Compare
@@ -746,6 +752,46 @@ describe('ConversationRepository', () => { | |||
}); | |||
}); | |||
|
|||
it('should add other self clients to mls group if user was event creator', () => { |
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.
🤩
@@ -2460,28 +2460,34 @@ export class ConversationRepository { | |||
}); | |||
} | |||
|
|||
// Self user is a creator of the event | |||
const isFromSelf = eventJson.from === this.userState.self().id; | |||
|
|||
// Self user joins again |
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.
Is this still relevant?
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.
nope, removed it 👌
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.
Looks solid. Great job 😎
When user receives
member-join
event, and it's also a creator of this event, it means that user has joined by himself, therefore we need to add its other clients to mls group if conversation is using mls protocol