Skip to content

docs(activity): clarify enable_email toggle scope and queue drain behaviour#14781

Open
miaulalala wants to merge 1 commit intomasterfrom
fix/activity-email-toggle-queue-behaviour
Open

docs(activity): clarify enable_email toggle scope and queue drain behaviour#14781
miaulalala wants to merge 1 commit intomasterfrom
fix/activity-email-toggle-queue-behaviour

Conversation

@miaulalala
Copy link
Copy Markdown
Contributor

Summary

  • Added a .. note:: block in the Admin settings section explaining that the Enable notification emails toggle only prevents new emails from being queued — it does not retroactively remove emails already in the oc_activity_mq table.
  • Added a new Troubleshooting entry for the scenario where emails continue to be sent after the toggle is disabled, with the fix (truncate oc_activity_mq).

Background

Two bugs were fixed in the Activity app (nextcloud/activity#2562):

  1. bulkReceive() email bypass (stable33+master): Group-share events went through a bulk DB path that accidentally bypassed the enable_email admin toggle and queued emails even when email was globally disabled. Now fixed.
  2. Queue drain after disabling the toggle (pre-existing, unfixed): MailQueueHandler — the background job that sends queued emails — does not re-check the toggle before sending. Emails already in the queue when the toggle is turned off will still be delivered. The only way to stop them immediately is to clear oc_activity_mq.

This documentation clarifies both behaviours for administrators.

🤖 Generated with Claude Code

…aviour

Signed-off-by: Anna Larch <anna@nextcloud.com>
@miaulalala miaulalala force-pushed the fix/activity-email-toggle-queue-behaviour branch from a8b4526 to 05ee6c2 Compare May 6, 2026 11:47
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

📖 Documentation Preview

🔍 Open preview →

📄 1 changed documentation page

Last updated: Wed, 06 May 2026 12:03:12 GMT

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant