From 29967d7e4ecafcc1465932c2a68ce564b103652b Mon Sep 17 00:00:00 2001
From: Nico <47644445+nicotsx@users.noreply.github.com>
Date: Wed, 25 Feb 2026 18:54:56 +0100
Subject: [PATCH] fix(discord): do not split each line into individual messages
 (#573)

Closes #519

<!-- This is an auto-generated comment: release notes by coderabbit.ai -->

## Summary by CodeRabbit

* **New Features**
  * Notification destinations can now be tested even when disabled, enabling configuration validation before activation

* **Improvements**
  * Discord notification formatting parameters optimized for enhanced message delivery

<!-- end of auto-generated comment: release notes by coderabbit.ai -->
---
 app/server/modules/notifications/builders/discord.ts      | 2 ++
 app/server/modules/notifications/notifications.service.ts | 6 +-----
 2 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/app/server/modules/notifications/builders/discord.ts b/app/server/modules/notifications/builders/discord.ts
index 681e9997..6ef68cf1 100644
--- a/app/server/modules/notifications/builders/discord.ts
+++ b/app/server/modules/notifications/builders/discord.ts
@@ -13,6 +13,8 @@ export const buildDiscordShoutrrrUrl = (config: Extract<NotificationConfig, { ty
 	let shoutrrrUrl = `discord://${webhookToken}@${webhookId}`;
 
 	const params = new URLSearchParams();
+
+	params.append("splitLines", "false");
 	if (config.username) {
 		params.append("username", config.username);
 	}
diff --git a/app/server/modules/notifications/notifications.service.ts b/app/server/modules/notifications/notifications.service.ts
index 73d48a44..4f4222d6 100644
--- a/app/server/modules/notifications/notifications.service.ts
+++ b/app/server/modules/notifications/notifications.service.ts
@@ -1,5 +1,5 @@
 import { eq, and } from "drizzle-orm";
-import { BadRequestError, ConflictError, InternalServerError, NotFoundError } from "http-errors-enhanced";
+import { BadRequestError, InternalServerError, NotFoundError } from "http-errors-enhanced";
 import { db } from "../../db/db";
 import {
 	notificationDestinationsTable,
@@ -229,10 +229,6 @@ const deleteDestination = async (id: number) => {
 const testDestination = async (id: number) => {
 	const destination = await getDestination(id);
 
-	if (!destination.enabled) {
-		throw new ConflictError("Cannot test disabled notification destination");
-	}
-
 	const decryptedConfig = await decryptSensitiveFields(destination.config);
 
 	const shoutrrrUrl = buildShoutrrrUrl(decryptedConfig);