Skip to content

Conversation

@rnons
Copy link
Contributor

@rnons rnons commented Jan 25, 2026

No description provided.

@rnons rnons requested a review from tulir January 25, 2026 05:54
Comment on lines 164 to 167
MemberActions: map[event.MemberAction]event.CapabilitySupportLevel{
event.MemberActionInvite: event.CapLevelFullySupported,
event.MemberActionKick: event.CapLevelFullySupported,
},
Copy link
Member

Choose a reason for hiding this comment

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

These should only be set in igCapsGroup and metaCapsGroup. I guess metaCapsWithE2E also needs to be split up to have groups separately

}
}
for _, userID := range evt.Leave {
memberChanges.MemberMap[metaid.MakeWAUserID(userID)] = bridgev2.ChatMember{
Copy link
Member

Choose a reason for hiding this comment

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

memberChanges.MemberMap.Set(bridgev2.ChatMember{...}) is preferred over manually setting map entries (that way the user ID doesn't need to be repeated)

@rnons rnons merged commit 50b3ddd into main Jan 27, 2026
11 checks passed
@rnons rnons deleted the rnons/plat-35084 branch January 27, 2026 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants