fix(ui): hide mark as unread for own messages#2590
fix(ui): hide mark as unread for own messages#2590Brazol wants to merge 2 commits intofeat/design-refreshfrom
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## feat/design-refresh #2590 +/- ##
====================================================
Coverage 65.32% 65.32%
====================================================
Files 429 428 -1
Lines 26137 26137
====================================================
+ Hits 17073 17075 +2
+ Misses 9064 9062 -2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
packages/stream_chat_flutter/lib/src/message_action/message_actions_builder.dart
Outdated
Show resolved
Hide resolved
| // If message is a parent message, it can be marked unread independent of | ||
| // other logic. | ||
| if (isParentMessage) { | ||
| messageActions.add(markUnreadAction()); | ||
| } | ||
| // If the message is in the channel view, only other user messages can be | ||
| // marked unread. |
There was a problem hiding this comment.
We should also update the inline doc now that we no longer check for currentUser in this if block. and also same for the parent message.
2 participants
Submit a pull request
Linear: FLU-463
in-depth Slack thread
CLA
Description of the pull request
Screenshots / Videos