payload-plugin-newsletter 0.16.1 → 0.16.2

package/CHANGELOG.md

@@ -1,3 +1,18 @@
+## [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

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));
@@ -1450,13 +1473,13 @@ var createBroadcastsCollection = (pluginConfig) => {
               return doc;
             }
             try {
-              const broadcastConfig = pluginConfig.providers?.broadcast;
+              const broadcastConfig = await getBroadcastConfig(req, pluginConfig);
               const resendConfig = pluginConfig.providers?.resend;
               if (!broadcastConfig && !resendConfig) {
-                req.payload.logger.error("No provider configured for sending");
+                req.payload.logger.error("No provider configured for sending in Newsletter Settings or environment variables");
                 return doc;
               }
-              if (broadcastConfig) {
+              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);
@@ -1491,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));
@@ -1537,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));