feat: Added unread message count badge for channels / teams#40603
feat: Added unread message count badge for channels / teams#40603devanshkansagra wants to merge 2 commits into
Conversation
|
Looks like this PR is not ready to merge, because of the following issues:
Please fix the issues and try again If you have any trouble, please check the PR guidelines |
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (5)
📜 Recent review details🧰 Additional context used📓 Path-based instructions (1)**/*.{ts,tsx,js}📄 CodeRabbit inference engine (.cursor/rules/playwright.mdc)
Files:
🧠 Learnings (4)📚 Learning: 2026-03-27T14:52:56.865ZApplied to files:
📚 Learning: 2026-05-06T12:21:44.083ZApplied to files:
📚 Learning: 2026-02-26T19:25:44.063ZApplied to files:
📚 Learning: 2026-02-26T19:25:44.063ZApplied to files:
🔇 Additional comments (5)
WalkthroughThe PR implements unread message counting for subscriptions with default "all messages" notification settings. It adds helper functions to detect eligible subscriptions, updates the subscription model with filtered query logic and a bulk unread increment method, and refines sidebar rendering to properly reflect unread state changes. ChangesAll-Messages Unread Counting
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Suggested labels
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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 |
Proposed changes (including videos or screenshots)
This PR adds unread count support for channels and teams when room notification settings are configured to count all messages.
Previously, channels/teams could be marked as unread visually through bold text or alert state, but no numeric unread badge was shown unless the subscription’s unread value was incremented. This made channel/team unread activity less visible than direct messages.
Demo
Screen.Recording.2026-05-19.at.1.45.11.AM.mov
Issue(s)
Steps to test or reproduce
Further comments
Summary by CodeRabbit