feat(notifications): skip non-account emails for inactive users#4744
feat(notifications): skip non-account emails for inactive users#4744SylvainChevalier wants to merge 1 commit into
Conversation
Users with is_active=False should only receive account-related emails (activation, password reset, email change confirmation, signup invite, deactivation notice). Filter inactive recipients out of: - the Notification scheduling/delivery pipeline - comment mention notifications - weekly top-comments digest - forecast auto-withdrawal warnings - multiple-choice option add/remove notifications - staff comment/key-factor report notifications - news category notebook publish notifications Fixes #4677 Co-authored-by: Sylvain <SylvainChevalier@users.noreply.github.com>
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@hlbmtc I know you're looking at emails, feel free to ignore if redundant. |
🚀 Preview EnvironmentYour preview environment is ready!
Details
ℹ️ Preview Environment InfoIsolation:
Limitations:
Cleanup:
|
|
This pull request has been automatically marked as stale because it has not had recent activity. It will not be closed automatically, but please consider updating it or closing it if it is no longer relevant. |
Closes #4677.
Users with
is_active=Falsenow only receive account-related emails (activation, password reset, email change confirmation, signup invite, deactivation notice). All other notification and digest emails are suppressed for inactive recipients, filtered at the user-queryset boundaries and in the notification scheduling/delivery pipeline.Generated with Claude Code