Stu Mason
Stu Mason

Activity

StuMason/cleanconnect
Client SaaS
TypeScript
Pull Request Merged

PR #195 merged: fix: give the admin watchlist notification a message

Those generic "New notification" entries in the admin bell are the Needs Attention summaries (PaymentWatchlistNotification).

Root cause

Confirmed on prod: every notification type stores a message except payment_watchlist (39 rows, no message). The bell falls back to "New notification" when data.message is empty — so only the admin watchlist ones look broken.

Fix

Add a count summary to toArray, e.g. "3 bookings and payments need attention." (or "1 booking or payment…" when singular).

  • Not test data, not from the comms sweep — pre-existing gap.
  • Existing 39 rows keep their stored (message-less) data; new watchlist notifications render properly. They age out / clear as you action them.

Test

Added an assertion that toArray() includes the summary message. Watchlist suite green (5).

+35
additions
-1
deletions
2
files changed