Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Push Notifications #100

Open
kitsunekyo opened this issue Mar 4, 2024 · 9 comments · May be fixed by #122
Open

Push Notifications #100

kitsunekyo opened this issue Mar 4, 2024 · 9 comments · May be fixed by #122
Assignees
Labels
enhancement New feature or request

Comments

@kitsunekyo
Copy link
Owner

Manche Benutzer würden gerne Benachrichtigungen erhalten sobald irgendeine Trainerin ein Praktikum einträgt.

@kitsunekyo kitsunekyo added the enhancement New feature or request label Mar 4, 2024
@kitsunekyo
Copy link
Owner Author

notes:

  • beim logout sollten subscriptions gelöscht(?) werden, sonst problematisch für shared clients
  • könnte man bei Änderungen von notifications permissions außerhalb der App (über browser settings) subscription triggern?

der ganze Userflow sollte noch überdacht werden.

@kitsunekyo
Copy link
Owner Author

aktuelle idee für den flow:

optin:

  1. user bekommt dialog modal "aktiviere push notifications + button"
  2. bei button click kommt browser prompt
  3. bei "erlauben" wird vom service worker eine subscription auf den pushManager gesetzt
  4. subscription json wird mit userId referenz in der db gespeichert
  5. server actions können an alle subscriptions via web-push senden

@kitsunekyo
Copy link
Owner Author

user sollten später jederzeit ihre meinung ändern können: granted > denied, denied > granted
subscription sollte auch funktionieren wenn von "denied" auf "granted" gewechselt wurde.

@kitsunekyo kitsunekyo changed the title Notifications Push Notifications Mar 17, 2024
@kitsunekyo
Copy link
Owner Author

beim inspecten von facebook ist mir aufgefallen, dass der serviceworker gelöscht wird beim logout. das würde das privacy problem möglicherweise lösen/verbessern.

@kitsunekyo
Copy link
Owner Author

@jnoggler schon mal sowas from scratch gebaut?

@kitsunekyo kitsunekyo linked a pull request Mar 17, 2024 that will close this issue
@jnoggler
Copy link
Collaborator

@kitsunekyo du meinst browser-based push notifications?
gebaut nicht, aber im rahmen unseres vergangenen projekts mal einiges an lösungen evaluiert

@kitsunekyo
Copy link
Owner Author

@kitsunekyo du meinst browser-based push notifications? gebaut nicht, aber im rahmen unseres vergangenen projekts mal einiges an lösungen evaluiert

wennst mal bock hast würden mich deine gedanken dazu interessieren. service-worker to browser is easy, aber web-push vom server getriggert is bissl ein rabbit hole mit dem subscription und VAPID key management^^

@kitsunekyo kitsunekyo self-assigned this Apr 6, 2024
@kitsunekyo
Copy link
Owner Author

wird wahrscheinlich knock.app werden. ist einfacher

@kitsunekyo
Copy link
Owner Author

https://knock.app/blog/how-to-send-browser-push-notifications-from-nodejs

brauch trotzdem vapid + das ganze zeug für webpush

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants