-
Notifications
You must be signed in to change notification settings - Fork 5
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: Update isReplyAllowed in Notifications according LegalHoldStatus [WPB-7425] #2874
feat: Update isReplyAllowed in Notifications according LegalHoldStatus [WPB-7425] #2874
Conversation
val isReplyAllowed = messages.first().run { | ||
degradedConversationNotified | ||
&& (legalHoldStatus == ConversationEntity.LegalHoldStatus.ENABLED && !legalHoldStatusChangeNotified) | ||
} |
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.
please add tests
Datadog ReportAll test runs ✅ 2 Total Test Services: 0 Failed, 2 Passed Test Services
|
Quality Gate passedIssues Measures |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## release/android-cycle-4.6 #2874 +/- ##
============================================================
Coverage ? 58.42%
Complexity ? 7
============================================================
Files ? 1185
Lines ? 46650
Branches ? 4432
============================================================
Hits ? 27256
Misses ? 17411
Partials ? 1983
Continue to review full report in Codecov by Sentry.
|
What's new in this PR?
Issues
When LegalHold activated and user was not notified about it, we need to disable "Reply" feature for notifications.
Causes (Optional)
Wasn't implemented
Solutions
Implement it.