Skip to content

Backend Support for Notification Settings Redesign - #76

Open
AdrianAndersen wants to merge 1 commit into
mainfrom
scheduled-webpush
Open

Backend Support for Notification Settings Redesign#76
AdrianAndersen wants to merge 1 commit into
mainfrom
scheduled-webpush

Conversation

@AdrianAndersen

@AdrianAndersen AdrianAndersen commented Jul 6, 2026

Copy link
Copy Markdown
Collaborator

This PR add support for the redesigned notification settings.

New Features

  • Scheduled web push notifications, I set up a new table with scheduled web push notifications that a cron job periodically checks to see if there are any due notifications. If a notification is due it sends it.
    • Scheduled notifications are regenerated when a user modifies their preferences, in addition to being scheduled/cancelled when a user books/unbooks a classe
  • A completely redesigned frontend, see PR in reservo-web
  • Stored Push Subscriptions now include notification grants, for booking, community and reminders
  • For slack reminders, we store a separate variable slack_reminders in the user preferences

One important note is that users will have to re-enable class reminders, since we now evaluate whether they want them or not with another variable. We could write a migration for it, but I think it is such a simple thing to do either manually in the DB or just each user for their own. Let me know what you think.

This PR depends on mathiazom/rezervo-web#160

Resolves #50

@AdrianAndersen AdrianAndersen self-assigned this Jul 6, 2026
@AdrianAndersen AdrianAndersen added the Feature A new feature label Jul 6, 2026
@AdrianAndersen AdrianAndersen moved this to Doing 🚀 in rezervo Jul 6, 2026
@AdrianAndersen AdrianAndersen moved this from Doing 🚀 to In Review 🕵️ in rezervo Jul 7, 2026
@AdrianAndersen
AdrianAndersen marked this pull request as ready for review July 7, 2026 14:52
@AdrianAndersen
AdrianAndersen requested review from Dentsor, mathiazom and redrei and removed request for mathiazom July 7, 2026 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature A new feature

Projects

Status: In Review 🕵️

Development

Successfully merging this pull request may close these issues.

Scheduled WebPush Notifications

1 participant