Skip to content

Notification Handling

Robert requested to merge feature/notifications into main

Ich überarbeite aktuell das Notification Handling.

  • Builtin-Benachrichtigungen gibt es bisher nicht explizit, sie sind nur ein aufwändiges Query. Zukünftig soll es sie explizit geben:
    • Hoffentlich deutliche Geschwindigkeitsverbesserung für angemeldete Benutzer:innen.
    • Möglichkeiten, sie einzeln zu verändern (beispielsweise einzeln "als gelesen markieren").
  • Alle Benachrichtigungen sollen in gleicher Art und Weise behandelt werden (E-Mail, Matrix, Builtin).
    • Das vereinfacht die Wartung des Benachrichtigungscodes.
    • Ungereimtheiten wie #775 (closed) sollten damit verschwinden.
  • In den Einstellungen soll es möglich sein, alle drei Benachrichtigungswege unabhängig voneinander zu konfigurieren.

Einstiegspunkt zum Lesen des neues Codes könnte grouprise.features.notifications.signals.send_notifications() sein. Dort wird das Signal empfangen, welches die Behandlung der Benachrichtigungen auslöst.

Was noch offen ist:

  • Builtin-Benachrichtigungen anzeigen
  • Builtin-Benachrichtigungen einzeln "als gelesen markieren" (ausgelagert in #789)
  • Benutzer-Einstellung "Benachrichtigungen" mit drei Checkboxen (E-Mail, Matrix, Builtin)
  • Klassen aus grouprise.features.notifications.signals verschieben (aufteilen auf verschiedene passende Module)
  • E-Mail-Notification-Klasse aus dem Core verschieben
  • veralteteten und ungenutzten E-Mail-Code möglichst entfernen
  • evtl. Code zum Anzeigen von Builtin-Benachrichtigungen nach features.builtin_inbox_notifications verschieben
Edited by Robert

Merge request reports