payload-plugin-newsletter 0.25.9 → 0.25.11

package/CHANGELOG.md

@@ -1,3 +1,12 @@
+## [0.25.11] - 2025-12-11
+
+### Fixed
+- Fixed LinkFeature configuration causing URLs to disappear in the Lexical editor link drawer
+  - The custom `fields` array was replacing Payload's default fields, removing the crucial `linkType` field
+  - Now uses default LinkFeature fields which include all required fields for the drawer UI to work correctly
+- Fixed TypeScript errors with Payload logger API (pino signature requires object first, then message)
+- Fixed `relationTo` type errors where `string | string[]` was passed to `findByID`
+
 ## [0.25.9] - 2025-08-19
 
 ### Fixed

package/dist/collections.cjs

@@ -754,25 +754,8 @@ var createEmailSafeFeatures = (additionalBlocks) => {
     (0, import_richtext_lexical.ItalicFeature)(),
     (0, import_richtext_lexical.UnderlineFeature)(),
     (0, import_richtext_lexical.StrikethroughFeature)(),
-    // Links with enhanced configuration
-    (0, import_richtext_lexical.LinkFeature)({
-      fields: [
-        {
-          name: "url",
-          type: "text",
-          required: true,
-          admin: {
-            description: "Enter the full URL (including https://)"
-          }
-        },
-        {
-          name: "newTab",
-          type: "checkbox",
-          label: "Open in new tab",
-          defaultValue: false
-        }
-      ]
-    }),
+    // Links - use default fields to ensure drawer UI works correctly
+    (0, import_richtext_lexical.LinkFeature)(),
     // Lists
     (0, import_richtext_lexical.OrderedListFeature)(),
     (0, import_richtext_lexical.UnorderedListFeature)(),
@@ -828,25 +811,8 @@ var createEmailLexicalEditor = (customBlocks = []) => {
       (0, import_richtext_lexical.ItalicFeature)(),
       (0, import_richtext_lexical.UnderlineFeature)(),
       (0, import_richtext_lexical.StrikethroughFeature)(),
-      // Links with enhanced configuration
-      (0, import_richtext_lexical.LinkFeature)({
-        fields: [
-          {
-            name: "url",
-            type: "text",
-            required: true,
-            admin: {
-              description: "Enter the full URL (including https://)"
-            }
-          },
-          {
-            name: "newTab",
-            type: "checkbox",
-            label: "Open in new tab",
-            defaultValue: false
-          }
-        ]
-      }),
+      // Links - use default fields to ensure drawer UI works correctly
+      (0, import_richtext_lexical.LinkFeature)(),
       // Lists
       (0, import_richtext_lexical.OrderedListFeature)(),
       (0, import_richtext_lexical.UnorderedListFeature)(),
@@ -1414,7 +1380,7 @@ async function getBroadcastConfig(req, pluginConfig) {
     }
     return pluginConfig.providers?.broadcast || null;
   } catch (error) {
-    req.payload.logger.error("Failed to get broadcast config from settings:", error);
+    req.payload.logger.error({ error: String(error) }, "Failed to get broadcast config from settings");
     return pluginConfig.providers?.broadcast || null;
   }
 }
@@ -1811,23 +1777,24 @@ async function populateBlockMediaFields(node, payload, config) {
       for (const field of blockConfig.fields) {
         if (field.type === "upload" && field.relationTo && node.fields[field.name]) {
           const fieldValue = node.fields[field.name];
+          const collectionName = Array.isArray(field.relationTo) ? field.relationTo[0] : field.relationTo;
           if (typeof fieldValue === "string" && fieldValue.match(/^[a-f0-9]{24}$/i)) {
             try {
               const media = await payload.findByID({
-                collection: field.relationTo,
+                collection: collectionName,
                 id: fieldValue,
                 depth: 0
               });
               if (media) {
                 node.fields[field.name] = media;
-                payload.logger?.info(`Populated ${field.name} for block ${blockType}:`, {
+                payload.logger?.info({
                   mediaId: fieldValue,
                   mediaUrl: media.url,
                   filename: media.filename
-                });
+                }, `Populated ${field.name} for block ${blockType}`);
               }
             } catch (error) {
-              payload.logger?.error(`Failed to populate ${field.name} for block ${blockType}:`, error);
+              payload.logger?.error({ error: String(error) }, `Failed to populate ${field.name} for block ${blockType}`);
             }
           }
         }
@@ -1839,23 +1806,24 @@ async function populateBlockMediaFields(node, payload, config) {
                 for (const arrayField of field.fields) {
                   if (arrayField.type === "upload" && arrayField.relationTo && arrayItem[arrayField.name]) {
                     const arrayFieldValue = arrayItem[arrayField.name];
+                    const arrayCollectionName = Array.isArray(arrayField.relationTo) ? arrayField.relationTo[0] : arrayField.relationTo;
                     if (typeof arrayFieldValue === "string" && arrayFieldValue.match(/^[a-f0-9]{24}$/i)) {
                       try {
                         const media = await payload.findByID({
-                          collection: arrayField.relationTo,
+                          collection: arrayCollectionName,
                           id: arrayFieldValue,
                           depth: 0
                         });
                         if (media) {
                           arrayItem[arrayField.name] = media;
-                          payload.logger?.info(`Populated array ${arrayField.name} for block ${blockType}:`, {
+                          payload.logger?.info({
                             mediaId: arrayFieldValue,
                             mediaUrl: media.url,
                             filename: media.filename
-                          });
+                          }, `Populated array ${arrayField.name} for block ${blockType}`);
                         }
                       } catch (error) {
-                        payload.logger?.error(`Failed to populate array ${arrayField.name} for block ${blockType}:`, error);
+                        payload.logger?.error({ error: String(error) }, `Failed to populate array ${arrayField.name} for block ${blockType}`);
                       }
                     }
                   }
@@ -1899,14 +1867,14 @@ async function populateRichTextUploads(content, payload) {
         });
         if (media) {
           node.value = media;
-          payload.logger?.info(`Populated rich text upload node:`, {
+          payload.logger?.info({
             mediaId: node.value,
             mediaUrl: media.url,
             filename: media.filename
-          });
+          }, "Populated rich text upload node");
         }
       } catch (error) {
-        payload.logger?.error(`Failed to populate rich text upload ${node.value}:`, error);
+        payload.logger?.error({ error: String(error) }, `Failed to populate rich text upload ${node.value}`);
       }
     }
     if (node.children && Array.isArray(node.children)) {
@@ -2310,7 +2278,7 @@ var createBroadcastsCollection = (pluginConfig) => {
             return doc;
           }
           if (operation === "update") {
-            req.payload.logger.info("Broadcast afterChange update hook triggered", {
+            req.payload.logger.info({
               operation,
               hasProviderId: !!doc.providerId,
               hasExternalId: !!doc.externalId,
@@ -2318,7 +2286,7 @@ var createBroadcastsCollection = (pluginConfig) => {
               publishStatus: doc._status,
               hasSubject: !!doc.subject,
               hasContent: !!doc.contentSection?.content
-            });
+            }, "Broadcast afterChange update hook triggered");
             try {
               const providerConfig = await getBroadcastConfig(req, pluginConfig);
               if (!providerConfig || !providerConfig.token) {
@@ -2405,10 +2373,10 @@ var createBroadcastsCollection = (pluginConfig) => {
                   if (JSON.stringify(doc.audienceIds) !== JSON.stringify(previousDoc?.audienceIds)) {
                     updates.audienceIds = doc.audienceIds?.map((a) => a.audienceId);
                   }
-                  req.payload.logger.info("Syncing broadcast updates to provider", {
+                  req.payload.logger.info({
                     providerId: doc.providerId,
                     updates
-                  });
+                  }, "Syncing broadcast updates to provider");
                   await provider.update(doc.providerId, updates);
                   req.payload.logger.info(`Broadcast ${doc.id} synced to provider successfully`);
                 } else {
@@ -2430,18 +2398,18 @@ var createBroadcastsCollection = (pluginConfig) => {
                   ...error.statusText
                 });
               } else if (typeof error === "string") {
-                req.payload.logger.error("Error is a string:", error);
+                req.payload.logger.error({ errorValue: error }, "Error is a string");
               } else if (error && typeof error === "object") {
-                req.payload.logger.error("Error is an object:", JSON.stringify(error, null, 2));
+                req.payload.logger.error({ errorValue: JSON.stringify(error, null, 2) }, "Error is an object");
               } else {
-                req.payload.logger.error("Unknown error type:", typeof error);
+                req.payload.logger.error({ errorType: typeof error }, "Unknown error type");
               }
-              req.payload.logger.error("Failed broadcast document (update operation):", {
+              req.payload.logger.error({
                 id: doc.id,
                 subject: doc.subject,
                 hasContent: !!doc.contentSection?.content,
                 contentType: doc.contentSection?.content ? typeof doc.contentSection.content : "none"
-              });
+              }, "Failed broadcast document (update operation)");
             }
           }
           return doc;
@@ -2487,7 +2455,7 @@ var createBroadcastsCollection = (pluginConfig) => {
                   ...error.details
                 });
               } else {
-                req.payload.logger.error(`Failed to send broadcast ${doc.id}:`, error);
+                req.payload.logger.error({ error: String(error) }, `Failed to send broadcast ${doc.id}`);
               }
               await req.payload.update({
                 collection: "broadcasts",
@@ -2530,7 +2498,7 @@ var createBroadcastsCollection = (pluginConfig) => {
                 ...error.details
               });
             } else {
-              req.payload.logger.error("Failed to delete broadcast from provider:", error);
+              req.payload.logger.error({ error: String(error) }, "Failed to delete broadcast from provider");
             }
           }
           return doc;