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:
|
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