Skip to content

feat: Add "Mark all as read" and unread message count indicators#4720

Merged
jamesarich merged 1 commit intomainfrom
fix/unreads
Mar 5, 2026
Merged

feat: Add "Mark all as read" and unread message count indicators#4720
jamesarich merged 1 commit intomainfrom
fix/unreads

Conversation

@jamesarich
Copy link
Copy Markdown
Collaborator

  • Add clearAllUnreadCounts to the packet repository and DAO.
  • Display a badge with the unread count on the messaging scroll-to-bottom FAB.
  • Add a "Mark as read" action to the contacts list.
  • Update SQL queries to exclude filtered packets from unread counts.
  • Improve the logic for scrolling to the first unread message in long conversations.
  • Reduce scroll debounce for better UI responsiveness.

- Add `clearAllUnreadCounts` to the packet repository and DAO.
- Display a badge with the unread count on the messaging scroll-to-bottom FAB.
- Add a "Mark as read" action to the contacts list.
- Update SQL queries to exclude filtered packets from unread counts.
- Improve the logic for scrolling to the first unread message in long conversations.
- Reduce scroll debounce for better UI responsiveness.
@jamesarich jamesarich requested a review from Copilot March 5, 2026 17:45
@github-actions github-actions bot added the bugfix PR tag label Mar 5, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0.00%. Comparing base (6a1a612) to head (7c6cd31).
⚠️ Report is 3 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@          Coverage Diff          @@
##            main   #4720   +/-   ##
=====================================
  Coverage   0.00%   0.00%           
=====================================
  Files          3       3           
  Lines        231     231           
  Branches      34      34           
=====================================
  Misses       231     231           

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

@jamesarich jamesarich added this pull request to the merge queue Mar 5, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Merged via the queue into main with commit b0258d0 Mar 5, 2026
7 checks passed
@jamesarich jamesarich deleted the fix/unreads branch March 5, 2026 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix PR tag

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants