payload-plugin-newsletter 0.16.0 → 0.16.2

package/CHANGELOG.md

@@ -1,3 +1,26 @@
+## [0.16.2] - 2025-07-27
+
+### Fixed
+- **Configuration Consistency** - Fixed broadcast operations to read from Newsletter Settings collection
+  - Broadcast create/update/delete operations now check Newsletter Settings first before falling back to env vars
+  - This matches the behavior of email operations (magic links, welcome emails)
+  - Resolves issue where broadcasts failed when env vars were missing despite settings being configured
+  - Added `getBroadcastConfig` utility for consistent configuration retrieval
+
+### Added
+- **Configuration Utilities** - New utilities for consistent provider configuration
+  - `getBroadcastConfig` - Gets Broadcast provider config from settings or env vars
+  - `getResendConfig` - Gets Resend provider config from settings or env vars
+  - Both utilities handle errors gracefully with fallback to env vars
+
+## [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

@@ -1162,6 +1162,29 @@ function wrapInEmailTemplate(content, preheader) {
 </html>`;
 }
 
+// src/utils/getBroadcastConfig.ts
+async function getBroadcastConfig(req, pluginConfig) {
+  try {
+    const settings = await req.payload.findGlobal({
+      slug: pluginConfig.settingsSlug || "newsletter-settings",
+      req
+    });
+    if (settings?.provider === "broadcast" && settings?.broadcastSettings) {
+      return {
+        apiUrl: settings.broadcastSettings.apiUrl || pluginConfig.providers?.broadcast?.apiUrl || "",
+        token: settings.broadcastSettings.token || pluginConfig.providers?.broadcast?.token || "",
+        fromAddress: settings.fromAddress || pluginConfig.providers?.broadcast?.fromAddress || "",
+        fromName: settings.fromName || pluginConfig.providers?.broadcast?.fromName || "",
+        replyTo: settings.replyTo || pluginConfig.providers?.broadcast?.replyTo
+      };
+    }
+    return pluginConfig.providers?.broadcast || null;
+  } catch (error) {
+    req.payload.logger.error("Failed to get broadcast config from settings:", error);
+    return pluginConfig.providers?.broadcast || null;
+  }
+}
+
 // src/collections/Broadcasts.ts
 var createBroadcastsCollection = (pluginConfig) => {
   const hasProviders = !!(pluginConfig.providers?.broadcast || pluginConfig.providers?.resend);
@@ -1402,9 +1425,9 @@ var createBroadcastsCollection = (pluginConfig) => {
         async ({ doc, operation, req }) => {
           if (!hasProviders || operation !== "create") return doc;
           try {
-            const providerConfig = pluginConfig.providers?.broadcast;
-            if (!providerConfig) {
-              req.payload.logger.error("Broadcast provider not configured");
+            const providerConfig = await getBroadcastConfig(req, pluginConfig);
+            if (!providerConfig || !providerConfig.token) {
+              req.payload.logger.error("Broadcast provider not configured in Newsletter Settings or environment variables");
               return doc;
             }
             const { BroadcastApiProvider: BroadcastApiProvider2 } = await Promise.resolve().then(() => (init_broadcast2(), broadcast_exports));
@@ -1439,6 +1462,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 = await getBroadcastConfig(req, pluginConfig);
+              const resendConfig = pluginConfig.providers?.resend;
+              if (!broadcastConfig && !resendConfig) {
+                req.payload.logger.error("No provider configured for sending in Newsletter Settings or environment variables");
+                return doc;
+              }
+              if (broadcastConfig && broadcastConfig.token) {
+                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
@@ -1446,9 +1514,9 @@ var createBroadcastsCollection = (pluginConfig) => {
         async ({ data, originalDoc, operation, req }) => {
           if (!hasProviders || !originalDoc?.providerId || operation !== "update") return data;
           try {
-            const providerConfig = pluginConfig.providers?.broadcast;
-            if (!providerConfig) {
-              req.payload.logger.error("Broadcast provider not configured");
+            const providerConfig = await getBroadcastConfig(req, pluginConfig);
+            if (!providerConfig || !providerConfig.token) {
+              req.payload.logger.error("Broadcast provider not configured in Newsletter Settings or environment variables");
               return data;
             }
             const { BroadcastApiProvider: BroadcastApiProvider2 } = await Promise.resolve().then(() => (init_broadcast2(), broadcast_exports));
@@ -1492,9 +1560,9 @@ var createBroadcastsCollection = (pluginConfig) => {
         async ({ doc, req }) => {
           if (!hasProviders || !doc?.providerId) return doc;
           try {
-            const providerConfig = pluginConfig.providers?.broadcast;
-            if (!providerConfig) {
-              req.payload.logger.error("Broadcast provider not configured");
+            const providerConfig = await getBroadcastConfig(req, pluginConfig);
+            if (!providerConfig || !providerConfig.token) {
+              req.payload.logger.error("Broadcast provider not configured in Newsletter Settings or environment variables");
               return doc;
             }
             const { BroadcastApiProvider: BroadcastApiProvider2 } = await Promise.resolve().then(() => (init_broadcast2(), broadcast_exports));
@@ -1507,48 +1575,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;
         }
       ]
     }