payload-plugin-newsletter 0.16.0 → 0.16.1

package/CHANGELOG.md

@@ -1,3 +1,11 @@
+## [0.16.1] - 2025-07-27
+
+### Fixed
+- **Critical Bug Fix** - Fixed afterChange hook placement for sending broadcasts
+  - The send hook was incorrectly placed in the `afterDelete` array instead of `afterChange`
+  - This prevented broadcasts from being sent when published
+  - Publishing broadcasts now correctly triggers the send functionality
+
 ## [0.16.0] - 2025-07-27
 
 ### Changed

package/dist/collections.cjs

@@ -1439,6 +1439,51 @@ var createBroadcastsCollection = (pluginConfig) => {
             req.payload.logger.error("Failed to create broadcast in provider:", error);
             return doc;
           }
+        },
+        // Hook to send when published
+        async ({ doc, operation, previousDoc, req }) => {
+          if (operation !== "update") return doc;
+          const wasUnpublished = !previousDoc?._status || previousDoc._status === "draft";
+          const isNowPublished = doc._status === "published";
+          if (wasUnpublished && isNowPublished && doc.providerId) {
+            if (doc.status === "sent" || doc.status === "sending") {
+              return doc;
+            }
+            try {
+              const broadcastConfig = pluginConfig.providers?.broadcast;
+              const resendConfig = pluginConfig.providers?.resend;
+              if (!broadcastConfig && !resendConfig) {
+                req.payload.logger.error("No provider configured for sending");
+                return doc;
+              }
+              if (broadcastConfig) {
+                const { BroadcastApiProvider: BroadcastApiProvider2 } = await Promise.resolve().then(() => (init_broadcast2(), broadcast_exports));
+                const provider = new BroadcastApiProvider2(broadcastConfig);
+                await provider.send(doc.providerId);
+              }
+              await req.payload.update({
+                collection: "broadcasts",
+                id: doc.id,
+                data: {
+                  status: "sending" /* SENDING */,
+                  sentAt: (/* @__PURE__ */ new Date()).toISOString()
+                },
+                req
+              });
+              req.payload.logger.info(`Broadcast ${doc.id} sent successfully`);
+            } catch (error) {
+              req.payload.logger.error(`Failed to send broadcast ${doc.id}:`, error);
+              await req.payload.update({
+                collection: "broadcasts",
+                id: doc.id,
+                data: {
+                  status: "failed" /* FAILED */
+                },
+                req
+              });
+            }
+          }
+          return doc;
         }
       ],
       // Sync updates with provider
@@ -1507,48 +1552,6 @@ var createBroadcastsCollection = (pluginConfig) => {
             req.payload.logger.error("Failed to delete broadcast from provider:", error);
           }
           return doc;
-        },
-        // Hook to send when published
-        async ({ doc, req }) => {
-          if (doc._status === "published" && doc.providerId) {
-            if (doc.status === "sent" || doc.status === "sending") {
-              return doc;
-            }
-            try {
-              const broadcastConfig = pluginConfig.providers?.broadcast;
-              const resendConfig = pluginConfig.providers?.resend;
-              if (!broadcastConfig && !resendConfig) {
-                req.payload.logger.error("No provider configured for sending");
-                return doc;
-              }
-              if (broadcastConfig) {
-                const { BroadcastApiProvider: BroadcastApiProvider2 } = await Promise.resolve().then(() => (init_broadcast2(), broadcast_exports));
-                const provider = new BroadcastApiProvider2(broadcastConfig);
-                await provider.send(doc.providerId);
-              }
-              await req.payload.update({
-                collection: "broadcasts",
-                id: doc.id,
-                data: {
-                  status: "sending" /* SENDING */,
-                  sentAt: (/* @__PURE__ */ new Date()).toISOString()
-                },
-                req
-              });
-              req.payload.logger.info(`Broadcast ${doc.id} sent successfully`);
-            } catch (error) {
-              req.payload.logger.error(`Failed to send broadcast ${doc.id}:`, error);
-              await req.payload.update({
-                collection: "broadcasts",
-                id: doc.id,
-                data: {
-                  status: "failed" /* FAILED */
-                },
-                req
-              });
-            }
-          }
-          return doc;
         }
       ]
     }