Merge pull request #29291 from acierto/notificationsend

Made "notification:send" action idempotent

.changeset/tame-sloths-brake.md

@@ -0,0 +1,5 @@
+---
+'@backstage/plugin-scaffolder-backend-module-notifications': patch
+---
+
+Made "notification:send" action idempotent

plugins/scaffolder-backend-module-notifications/src/actions/sendNotification.ts

@@ -133,9 +133,14 @@ export function createSendNotificationAction(options: {
       };
 
       try {
-        await notifications.send({
-          recipients: notificationRecipients,
-          payload,
+        await ctx.checkpoint({
+          key: `send.notification.${payload.title}`,
+          fn: async () => {
+            await notifications.send({
+              recipients: notificationRecipients,
+              payload,
+            });
+          },
         });
       } catch (e) {
         ctx.logger.error(`Failed to send notification: ${e}`);