Show muted icon on channel items for DMs with muted users#6449
Conversation
PR checklist ✅All required conditions are satisfied:
🎉 Great job! This PR is ready for review. |
SDK Size Comparison 📏
|
eff5f3a to
63d4889
Compare
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (5)
WalkthroughChannelListViewModel now incorporates global user mutes into channel-item creation; ChannelItemState gains ChangesUser-level mute state for direct message channels
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 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 |
|
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
|
🚀 Available in v7.2.0 |


Goal
In the channel list, we are currently displaying the muted icon only when the channel is muted. However, for direct messages, we should look at the muted users too.
We're basically going back to #6302
Closes #6442
Implementation
Check user mutes to decide whether to show the muted icon on items representing direct messages channels
🎨 UI Changes
Same as #6302
Testing
Mute a user and then check that the muted icon is shown on the corresponding direct message channel item
Summary by CodeRabbit
Bug Fixes
Tests