Skip to content

Enable notifications by default for new and existing users - #181

Merged
tiankaima merged 1 commit into
mainfrom
feat/notifications-default-on
Sep 23, 2026
Merged

tiankaima merged 1 commit into
mainfrom
feat/notifications-default-on

Conversation

@tiankaima

Copy link
Copy Markdown
Member

Summary

  • Turn on class, homework, Young Community, and todo reminders by default for new users when they log in.
  • Upgrade the current production SQLite database from version 5 to 6 once, enabling all four reminders for existing users and creating settings for users without a row. Users without credentials, or already paused for reauthorization, remain paused until login.
  • Preserve each user's later opt-out across token refreshes and future deployments.

Verification

  • go test ./...
  • make check-scripts
  • Production database inspected read-only: version 5, 155 users, 24 credentials, 2 existing notification settings rows.

The production Bot still logs intermittent 401 responses from Young Community notifications and 503 responses from homework notifications. This change does not alter those upstream responses.

@tiankaima
tiankaima merged commit b7d5c87 into main Sep 23, 2026
3 checks passed
@tiankaima
tiankaima deleted the feat/notifications-default-on branch September 23, 2026 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant