Notifikationen an User für abgeschlossene Tasks about madek

[phoettli]

a) verhältnismässig einfach zu implementieren

b) sehr umfangreicher Aufwand
Mailservice ggf. von leihs übernehmen

Vorgehen:

• Workshop für Refinement nach Workshops Fileservice

Grobschätzunbg Aufwand: 1 Mann-Quartal (DB, Backend und Frontend, Deploy und CI (Fullstack))

[nimaai]

Notifikationsziel
User => hinterlegte Adresse, i.d.R. ZHdK-Adresse (keine Änderung möglich?)
Delegation: Notifikationsadresse setzen, (mehrere E-Mails möglich) oder automatisch alle in Delegation

bitte definieren/präzisieren @phoettli "keine Änderungen möglich?"
"Notifikationsadresse setzen, (mehrere E-Mails möglich) oder automatisch alle in Delegation" wenn alle in Delegation, welche Email-Adresse vom User?

[nimaai]

Mail-Zusammenfassungen
alles

@phoettli was bedeutet das genau?

[nimaai]

Frage: angenommen die Email-Frequenz ist auf "weekly" eingestellt und es kamen in den letzten 7 Tagen keine neue Notifikationen dazu: sollte trotzdem jede Woche eine Email verschickt werden auch wenn nur mit "Es sind keine neuen Notifikationen vorhanden"?

[phoettli]

Frage: angenommen die Email-Frequenz ist auf "weekly" eingestellt und es kamen in den letzten 7 Tagen keine neue Notifikationen dazu: sollte trotzdem jede Woche eine Email verschickt werden auch wenn nur mit "Es sind keine neuen Notifikationen vorhanden"?

@nimaai ich würde gar keine senden.
(Für die anderen Fragen erstelle ich Mockups...)

[nimaai]

based on the existing specification this is my first draft of the tables:

                             Table "public.notifications"
     Column      |           Type           | Collation | Nullable |      Default      
-----------------+--------------------------+-----------+----------+-------------------
 id              | uuid                     |           | not null | gen_random_uuid()
 user_id         | uuid                     |           | not null | 
 is_acknowledged | boolean                  |           | not null | false
 content         | text                     |           | not null | 
 email_id        | uuid                     |           |          | 
 created_at      | timestamp with time zone |           | not null | now()
 updated_at      | timestamp with time zone |           | not null | now()
Indexes:
    "notifications_pkey" PRIMARY KEY, btree (id)
Triggers:
    update_updated_at_column_of_notifications BEFORE UPDATE ON notifications FOR EACH ROW WHEN (old.* IS DISTINCT FROM new.*) EXECUTE FUNCTION update_updated_at_column()

NOTE: email_id will be a FK to emails table

                                       Table "public.notifications_settings"
            Column            |           Type           | Collation | Nullable |             Default              
------------------------------+--------------------------+-----------+----------+----------------------------------
 id                           | uuid                     |           | not null | gen_random_uuid()
 user_id                      | uuid                     |           | not null | 
 deliver_via_email            | boolean                  |           | not null | false
 deliver_via_ui               | boolean                  |           | not null | false
 deliver_via_email_regularity | character varying        |           | not null | 'immediately'::character varying
 created_at                   | timestamp with time zone |           | not null | now()
 updated_at                   | timestamp with time zone |           | not null | now()
Indexes:
    "notifications_settings_pkey" PRIMARY KEY, btree (id)
Check constraints:
    "check_email_regularity_value" CHECK (deliver_via_email_regularity::text = ANY (ARRAY['immediately'::character varying, 'daily'::character varying, 'weekly'::character varying]::text[]))
Triggers:
    update_updated_at_column_of_notifications_settings BEFORE UPDATE ON notifications_settings FOR EACH ROW WHEN (old.* IS DISTINCT FROM new.*) EXECUTE FUNCTION update_updated_at_column()

@DrTom what is your first impression?

[phoettli]

Mail-Zusammenfassungen
alles
@nimaai ich habs oben korrigiert. Ich glaube ich meinte da: "alles einzelne Mails".
Und die Zusammenfassungen habe ich um wöchentlich ergänzt

[DrTom]

@DrTom what is your first impression?

Looks good.

At first glance deliver_via_email_regularity might have been some numeric value. But maybe it is better this way. Depends when it should be send.

[nimaai]

@phoettli was ist die idee bei wöchentlicher zustellung: pro notifikation je eine email oder zusammengefasst in einer einzigen?

[phoettli]

Mail-Zusammenfassungen

• 1x täglich (default)
• 1x wöchentlich
• alles einzeln (keine Zusammenfassung)
• Verantwortlichkeit und Upload separat
• ev. später weitere?

Mein Vorschlag (wie oben) die Optionen:

• täglich und wöchentlich als Zusammenfassung
• alles einzeln, wenn das jemand mag, oder so wenig Infos bekommt, dass das passt
• ev. noch Übertragung der Verantwortlichkeit und abgeschlossene Uploads separat
• wohl sowieso separat würden Admin-Infos wie Personen-Kollisionen (wenn neue User angelegt werden) und allenfalls weitere

oder was denkst du @nimaai ? bzw. was wäre praktikabel?

[nimaai]

@nimaai to consider irt to auto-delete trigger:
what about ui notifications which have not been acknowledged within the time period?

• the notifications should be deleted regardless.

[nimaai]

• @nimaai @urs-van-binsbergen the notifications will always be displayed to the user; it means no option to disable ui notifications.

[nimaai]

• @phoettli @urs-van-binsbergen default email frequency option should be "daily summary" not "immediately" to prevent spamming.

[nimaai]

• @nimaai auto-delete notifications after 6 months

[nimaai]

• @nimaai changing of the notification settings should affect only the future notifications ??