-
Notifications
You must be signed in to change notification settings - Fork 116
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
Remove Room moderation feature flag #2678
Remove Room moderation feature flag #2678
Conversation
📱 Scan the QR code below to install the build (arm64 only) for this PR. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #2678 +/- ##
===========================================
- Coverage 73.56% 73.55% -0.02%
===========================================
Files 1458 1458
Lines 35222 35207 -15
Branches 6766 6765 -1
===========================================
- Hits 25912 25897 -15
+ Misses 5787 5786 -1
- Partials 3523 3524 +1 ☔ View full report in Codecov by Sentry. |
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.
LGTM, can you add a file for towncrier please?
val isDm = room.isDm && room.isEncrypted | ||
return isRoomModerationEnabled && !isDm && (canBan() || canKick()) | ||
return !isDm && (canBan() || canKick()) |
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.
Out of scope of this PR, but the condition room.isEncrypted
above is a bit strange.
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.
At this point I'm not even sure what a DM should be. It started as a direct room with 2 users and encryption, but I'm not sure if we dropped the encryption check or we still use it...
Maestro died at the same point twice, it seems like the verification screen takes way too long to load... |
|
Type of change
Content
Removes the feature flag for room moderation, to make this feature available to everyone.
Motivation and context
The feature is complete and validated.
Screenshots / GIFs
Tests
Tested devices
Checklist