Remove Notification Triggers / find alternative #1881

New Issue

dpschen · 2021-12-03T14:50:52Z

dpschen commented

2021-12-03 14:50:52 +00:00

See this tweet for more infos:
https://twitter.com/ChromiumDev/status/1466749116990599171

See this tweet for more infos: https://twitter.com/ChromiumDev/status/1466749116990599171

dpschen added the

area/internal-code

label 2021-12-03 14:50:59 +00:00

konrad commented

2021-12-03 16:40:05 +00:00

I think we should pursue showing notifications for everyone, not only behind a chrome origin trial. That would require some kind of setInterval that checks every minute what reminders are due and then show them. The api supports querying tasks with reminders after a certain date, maybe that'd be the easiest to achieve this (poll it every minute for the reminders in the next minute). Would have the nice benefit to show reminders created on other clients or in another tab but increase the load on the api.

I think we should pursue showing notifications for everyone, not only behind a chrome origin trial. That would require some kind of `setInterval` that checks every minute what reminders are due and then show them. The api supports querying tasks with reminders after a certain date, maybe that'd be the easiest to achieve this (poll it every minute for the reminders in the next minute). Would have the nice benefit to show reminders created on other clients or in another tab but increase the load on the api.

👍 1

konrad commented

2021-12-03 16:42:57 +00:00

Another option would be to move reminders entirely to the api and let it push reminders via a websocket when due. The api already does something similar for reminders via email, that could be reused.

We'd need to build the websocket thing first, but I'd like to have one for the notifications at some point anyway.

Another option would be to move reminders entirely to the api and let it push reminders via a websocket when due. The api already does something similar for reminders via email, that could be reused. We'd need to build the websocket thing first, but I'd like to have one for the notifications at some point anyway.

dpschen commented

2021-12-04 14:36:49 +00:00

I think we should have an offline fallback. Other than that: agree =)

konrad referenced this issue from a commit

2024-02-07 14:00:33 +00:00

chore(deps): update dependency autoprefixer to v10.4.6 (#1881)

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: vikunja/vikunja#1881