Notifier - Githubissues

JM-Lemmi / ical-relay

Relay ical urls and edit them on the fly with different modules.

GNU Affero General Public License v3.0

9 stars 2 forks source link

Notifier #233

Closed JM-Lemmi closed 2 months ago

JM-Lemmi commented 9 months ago

Todo:

[x] develop strategy for lite mode and connection to ical relay
[x] own cmd, migrate old functions over
[x] use db datastore
[x] system tests
[x] ci building
[x] RSS feed output
[x] Frontend viewer
[x] Frontend subscription

JM-Lemmi commented 2 months ago

@Sarsum do you want to add a test for DB based ical-notifier?

Sarsum commented 2 months ago

@JM-Lemmi yes, we should add it. However, I am not able to implement it during the next two'ish weeks. So I'd suggest to do that in another PR.

JM-Lemmi commented 2 months ago

Ich würde die Recipients in der DB noch angehen, damit ist der notifier nämlich fertig.

JM-Lemmi commented 2 months ago

Warum sind denn die notifier recipients in einer extra tabelle mit den notifiern verknüpft, wenn sie eh unique sind. kann dann nicht die referenz direkt in der recipients table sein?

Sarsum commented 2 months ago

So ist es aktuell möglich, dass ein Recipient mehrere Notifier abonniert haben kann. Im Umkehrschluss könnten wir dann auch irgendwann über einen Unsubscribe Link die Möglichkeit bieten spezifische Notifier zu deabonnieren.