Notification Handling
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