Skip to content

Room list notifications tweaks - #6106

Draft
stefanceriu wants to merge 3 commits into
developfrom
stefan/roomListNotifications
Draft

Room list notifications tweaks#6106
stefanceriu wants to merge 3 commits into
developfrom
stefan/roomListNotifications

Conversation

@stefanceriu

@stefanceriu stefanceriu commented Sep 4, 2026

Copy link
Copy Markdown
Member
  • Remove the "Current" room list activity visibility feature flag option by merging it with "Show"
  • Introduce a new "Show unread notification count" feature flag to display the number of unread messages on the notification (green) badges.

Fixes #6101

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

PR Checks

  • ⚠️ You seem to have made changes to views. Please consider adding screenshots.

@stefanceriu stefanceriu added the pr-wip for anything that isn't ready to ship and will be enabled at a later date label Sep 4, 2026
@codecov

codecov Bot commented Sep 4, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 93.54839% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.98%. Comparing base (4355307) to head (d93c1f8).
⚠️ Report is 1 commits behind head on develop.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...perOptionsScreen/View/DeveloperOptionsScreen.swift 0.00% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #6106   +/-   ##
========================================
  Coverage    74.98%   74.98%           
========================================
  Files          829      829           
  Lines        58857    58875   +18     
========================================
+ Hits         44131    44149   +18     
  Misses       14726    14726           
Flag Coverage Δ
unittests 65.44% <93.54%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-wip for anything that isn't ready to ship and will be enabled at a later date

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Task] Align room list notification decorations

1 participant