From 04dfff4ebdede769f93572804b583f690e3122a2 Mon Sep 17 00:00:00 2001 From: LeC-D Date: Fri, 27 Mar 2026 14:19:27 -0400 Subject: [PATCH] fix(notifications): remove redundant defaultValue prop and fix incident URL in telegram provider - Remove redundant 'defaultValue' prop on Telegram chat ID Controller in create/index.tsx; the form already initialises via 'defaultValues: defaults' at the useForm level, making the per-Controller prop unnecessary and inconsistent with the reviewer's expectation. - Fix 'View Incident' URL in telegram.ts: was pointing to the infrastructure details page (/infrastructure/:id) instead of the incidents page (/incidents/:id). --- client/src/Pages/Notifications/create/index.tsx | 1 - .../service/infrastructure/notificationProviders/telegram.ts | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/client/src/Pages/Notifications/create/index.tsx b/client/src/Pages/Notifications/create/index.tsx index 0dbecfef2..85568f8bd 100644 --- a/client/src/Pages/Notifications/create/index.tsx +++ b/client/src/Pages/Notifications/create/index.tsx @@ -198,7 +198,6 @@ const NotificationsCreatePage = () => { ( View Incident`); + lines.push(`View Incident`); } return lines.join("\n");