From 30e644a0014f431e42f14260c9714223a4884ab4 Mon Sep 17 00:00:00 2001 From: Patrick Niebeling Date: Wed, 30 Sep 2026 11:02:04 +0200 Subject: [PATCH] Condense each moved card to one sentence in the mail Drops the workflow name; the mail only says which card moved where. Bumps to 35.2.1. --- appinfo/info.xml | 2 +- lib/Service/NotificationMailer.php | 9 +++------ package.json | 2 +- 3 files changed, 5 insertions(+), 8 deletions(-) diff --git a/appinfo/info.xml b/appinfo/info.xml index f4b7df0..f7e79e6 100644 --- a/appinfo/info.xml +++ b/appinfo/info.xml @@ -24,7 +24,7 @@ This app lets every user configure Deck automation workflows for the boards they A background job periodically evaluates all enabled workflows and moves cards whose due date has passed, using Deck's own internal PHP classes only (no HTTP/OCS calls). ]]> - 35.2.0 + 35.2.1 agpl Patrick Niebeling WorkflowDeckAutomation diff --git a/lib/Service/NotificationMailer.php b/lib/Service/NotificationMailer.php index a6c17ce..572f279 100644 --- a/lib/Service/NotificationMailer.php +++ b/lib/Service/NotificationMailer.php @@ -100,14 +100,11 @@ class NotificationMailer { private function addCard(IEMailTemplate $template, Workflow $workflow, Card $card, array $targets): void { // Single-argument addBodyText() runs htmlspecialchars() itself, so // these card-, workflow-, board- and stack-titles (user input) are - // safe as-is. The card description is deliberately left out. + // safe as-is. The card description and the workflow's name are + // deliberately left out — the mail only says what moved where. $template->addBodyText(sprintf( - 'Die Karte „%1$s“ wurde verschoben, weil sie überfällig ist (Workflow „%2$s“).', + '„%1$s“ (Board „%2$s“) ist überfällig und wurde von „%3$s“ nach „%4$s“ verschoben.', $card->getTitle(), - $workflow->getTitle(), - )); - $template->addBodyText(sprintf( - 'Board „%1$s“: von „%2$s“ nach „%3$s“ verschoben.', $targets['board'], $targets['sourceStack'], $targets['targetStack'], diff --git a/package.json b/package.json index d617ced..7d6ecf6 100644 --- a/package.json +++ b/package.json @@ -1,6 +1,6 @@ { "name": "workflow_deck_automation", - "version": "35.2.0", + "version": "35.2.1", "private": true, "type": "module", "description": "Automates moving overdue Deck cards between stacks, configurable per user.",