Skip to content

History / Notifications Developer Guide

Revisions

  • Notifications: new wiki pair for discussion #55 New pages (app commits bbd49c50 + 75803509): - Notifications (user) - Notifications-Developer-Guide The user page leads with what the bell does NOT do, because the four noise rules are the feature -- a bell that fills faster than it can be read is dead while still appearing to work. The dev guide records two things worth keeping: 1. The bell is a SUBSCRIBER to WorkflowEngine::dispatch(), not a second instrumentation layer. ~32 event types already fired from 48 call sites, so adding a notification type must never mean editing call sites. 2. EVERY failure mode in this feature is silent. Suppressed, rejected, thrown-and-swallowed and correctly-quiet all look like an empty table, so every rule needs a POSITIVE control and not just a negative one. The four bugs found during the build are written up for the same reason -- three of them presented as "notifications just never arrive". Also records three pre-existing broken ticket deep links (tickets/?id= in escalate_ticket.php and engine.php, tickets/?ticket= in forms/approvals.php). inbox.js reads only ?ticket_id=, so those return HTTP 200 and never open the ticket. Verified present, not fixed. Every named file verified to exist; every link verified to resolve.

    @edmozley edmozley committed Aug 12, 2026