feat: daily reminder mail for erroneous tasks (admins + WF owners, per-user opt-out)#39
Open
JeJan wants to merge 1 commit into
Open
feat: daily reminder mail for erroneous tasks (admins + WF owners, per-user opt-out)#39JeJan wants to merge 1 commit into
JeJan wants to merge 1 commit into
Conversation
…ner, Opt-out je Benutzer, ADR-009)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Daily e-mail digest of all workflow tasks in an error state (see ADR-009):
wf-admin+EMAIL_RECEIVERS_ERRONEOUS_TASKS) receive a global overview across all workflows; WF owners (BPMN propertywf-owner) only receive the errors of their own workflows. Users who are both get the admin mail only.* NEW *; each entry shows since when the task has been erroneous (new columnerror_at).Configuration
EMAIL_ERRONEOUS_TASKS_REMINDER_CRON(default0 7 * * *, empty = disabled; invalid expressions are logged and disable only this job)EMAIL_RECEIVERS_ERRONEOUS_TASKS(existing list, always receives the global digest)Migration
e8b3d5c7a9f1:workflow_instance_tasks.error_at+error_reported_at,workflow_users.receive_error_task_reminder(default on). Runs on app startup as usual.Tests
10 new tests (
test_erroneous_reminder.py): new/known cycle, recovery reset, owner scoping, admin dedup, opt-out incl. static-list exemption, partial send failures, cancelled instances. Full backend suite green, frontend typecheck green. Verified end-to-end against a local SMTP catcher (Mailpit).Note
The migration shares its parent (
d4f7a1e9c2b6) with the one from the form-template PR — whoever merges second must rebase their migration or create an Alembic merge revision.