Skip to content

Notification digest: group multiple notifications on the same target into one line #117

Description

@chubes4

Context

Audit follow-up (notification system polish). Not a bug — a quality improvement.

The unread-notification digest (inc/notifications/email.php) lists each notification as its own preview line. Five replies on one of your topics = five separate line items. Now that we nudge once per notification (#112), a busy thread still produces a noisy digest.

Proposal

Group preview items by target (item_id + type) when building the digest body, e.g. "3 new replies on your topic X" instead of three lines. Keep the total unread count accurate; only the preview rendering collapses.

  • Pure change in ec_notifications_email_build_digest() (already side-effect-free / unit-testable).
  • No schema change.

Parent epic: Extra-Chill/extrachill-community#82.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions