@overmap-ai/core 1.0.44-tiptap.5 → 1.0.44-tiptap.7

package/dist/overmap-core.js

@@ -1684,8 +1684,14 @@ const selectAttachmentsOfComponentByType = restructureCreateSelectorWithArgs(
     [selectAllComponentAttachments, (_state, componentId) => componentId],
     (attachments, componentId) => {
       const attachmentsOfComponent = attachments.filter(({ component_id }) => componentId === component_id);
-      const fileAttachments = attachmentsOfComponent.filter(({ file_type }) => !file_type.startsWith("image/"));
-      const imageAttachments = attachmentsOfComponent.filter(({ file_type }) => file_type.startsWith("image/"));
+      const fileAttachments = attachmentsOfComponent.filter(
+        // this null check here is necessary, there are cases where file_type is null or undefined
+        ({ file_type }) => !file_type || !file_type.startsWith("image/")
+      );
+      const imageAttachments = attachmentsOfComponent.filter(
+        // this null check here is necessary, there are cases where file_type is null or undefined
+        ({ file_type }) => file_type && file_type.startsWith("image/")
+      );
       return { fileAttachments, imageAttachments };
     }
   )
@@ -1950,8 +1956,14 @@ const selectAttachmentsOfComponentTypeByType = restructureCreateSelectorWithArgs
       const attachmentsOfComponent = attachments.filter(
         ({ component_type_id }) => component_type_id === componentTypeId
       );
-      const fileAttachments = attachmentsOfComponent.filter(({ file_type }) => !file_type.startsWith("image/"));
-      const imageAttachments = attachmentsOfComponent.filter(({ file_type }) => file_type.startsWith("image/"));
+      const fileAttachments = attachmentsOfComponent.filter(
+        // this null check here is necessary, there are cases where file_type is null or undefined
+        ({ file_type }) => !file_type || !file_type.startsWith("image/")
+      );
+      const imageAttachments = attachmentsOfComponent.filter(
+        // this null check here is necessary, there are cases where file_type is null or undefined
+        ({ file_type }) => file_type && file_type.startsWith("image/")
+      );
       return { fileAttachments, imageAttachments };
     }
   )
@@ -2251,8 +2263,14 @@ const selectAttachmentsOfIssueByType = restructureCreateSelectorWithArgs(
     [selectIssueAttachments, (_state, issueId) => issueId],
     (attachments, issueId) => {
       const attachmentsOfComponent = attachments.filter(({ issue_id }) => issue_id === issueId);
-      const fileAttachments = attachmentsOfComponent.filter(({ file_type }) => !file_type.startsWith("image/"));
-      const imageAttachments = attachmentsOfComponent.filter(({ file_type }) => file_type.startsWith("image/"));
+      const fileAttachments = attachmentsOfComponent.filter(
+        // this null check here is necessary, there are cases where file_type is null or undefined
+        ({ file_type }) => !file_type || !file_type.startsWith("image/")
+      );
+      const imageAttachments = attachmentsOfComponent.filter(
+        // this null check here is necessary, there are cases where file_type is null or undefined
+        ({ file_type }) => file_type && file_type.startsWith("image/")
+      );
       return { fileAttachments, imageAttachments };
     }
   )
@@ -4101,6 +4119,10 @@ class AttachmentService extends BaseApiService {
         ...fileProps
       }
     });
+    promise.catch((error2) => {
+      this.client.store.dispatch(removeIssueAttachment(offlineAttachment.offline_id));
+      throw error2;
+    });
     return [offlineAttachment, promise];
   }
   async addComponentAttachment(attachmentPayload) {
@@ -4131,6 +4153,10 @@ class AttachmentService extends BaseApiService {
         ...fileProps
       }
     });
+    promise.catch((error2) => {
+      this.client.store.dispatch(removeComponentAttachment(offlineAttachment.offline_id));
+      throw error2;
+    });
     return [offlineAttachment, promise];
   }
   async addComponentTypeAttachment(attachmentPayload) {
@@ -4150,7 +4176,7 @@ class AttachmentService extends BaseApiService {
     const promise = this.enqueueRequest({
       description: "Create attachment",
       method: HttpMethod.POST,
-      url: `/component_types/${component_type_id}/attach/`,
+      url: `/types/${component_type_id}/attach/`,
       blocks: [offline_id, component_type_id],
       blockers: [file_sha1],
       payload: {
@@ -4161,6 +4187,10 @@ class AttachmentService extends BaseApiService {
         ...fileProps
       }
     });
+    promise.catch((error2) => {
+      this.client.store.dispatch(removeComponentTypeAttachment(offlineAttachment.offline_id));
+      throw error2;
+    });
     return [offlineAttachment, promise];
   }
   /** the outer Promise is needed to await the hashing of each file, which is required before offline use. If wanting to
@@ -12530,7 +12560,7 @@ const ImageCard = memo((props) => {
       gap: "0",
       ...rest,
       children: [
-        !file && /* @__PURE__ */ jsx(Flex, { width: "100%", height: "100%", align: "center", justify: "center", position: "absolute", children: /* @__PURE__ */ jsx(Spinner, {}) }),
+        !file && !error2 && /* @__PURE__ */ jsx(Flex, { width: "100%", height: "100%", align: "center", justify: "center", position: "absolute", children: /* @__PURE__ */ jsx(Spinner, {}) }),
         /* @__PURE__ */ jsx(Inset, { className: styles$4.ImageInset, ref: imageInsetRef, clip: "padding-box", side: "y", pb: "0", children: file && !error2 && /* @__PURE__ */ jsx("img", { className: styles$4.Image, src: URL.createObjectURL(file), alt: alt ?? file.name }) }),
         /* @__PURE__ */ jsx(
           OvermapItem,