Skip to content

Conversation

@Akinator31
Copy link
Member

This pull request introduces support for role-targeted reminders and user opt-outs in the reminders system. It adds the ability to specify target roles when creating reminders, ensures that only users who have not opted out of reminders for a given role are notified, and provides a new command for managing reminder subscriptions. Several SQL queries and command registration logic are updated to support these features.

Role-targeted reminders and opt-out logic:

  • Added support for specifying target_roles when creating reminders, including updating the relevant SQL queries and command options to handle this new field. This allows reminders to be sent to all users with specific roles.
  • Implemented logic in spawn_reminder to mention only users with the targeted roles who have not opted out, using the new get_targeted_mentions function.
  • Added new SQL queries for managing reminder opt-outs, including inserting, selecting, and deleting opt-out records.

User interface and command changes:

  • Updated confirmation and reminder messages to include information about targeted roles, with new translation keys for different scenarios (with/without content, with/without roles).
  • Registered the new ReminderSubscriptionCommand to allow users to manage their reminder role subscriptions.

Other changes:

  • Cleaned up unused or obsolete queries, such as removing the old staff alert update query.
  • Minor improvements to time formatting and imports.

These changes collectively enhance the flexibility of the reminders system, enabling more granular control over which users receive reminders based on their roles and personal preferences.

@Akinator31 Akinator31 self-assigned this Jan 12, 2026
@Akinator31 Akinator31 linked an issue Jan 12, 2026 that may be closed by this pull request
@Akinator31 Akinator31 merged commit 654943b into main Jan 12, 2026
6 checks passed
@Akinator31 Akinator31 deleted the 324-ping-role-via-reminder-command branch January 12, 2026 19:00
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.

Ping role via reminder command

2 participants