From 8a1309536b76ba86872ca2e78aa695d9fd80c8cc Mon Sep 17 00:00:00 2001
From: MohamedBassem <me@mbassem.com>
Date: Sat, 5 Oct 2024 20:08:50 +0000
Subject: feature: Allow attaching custom banners to notes. Fixes: #106

---
 apps/web/components/dashboard/preview/AttachmentBox.tsx | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

(limited to 'apps/web/components/dashboard/preview/AttachmentBox.tsx')

diff --git a/apps/web/components/dashboard/preview/AttachmentBox.tsx b/apps/web/components/dashboard/preview/AttachmentBox.tsx
index b2460165..a8eaf0f4 100644
--- a/apps/web/components/dashboard/preview/AttachmentBox.tsx
+++ b/apps/web/components/dashboard/preview/AttachmentBox.tsx
@@ -27,7 +27,11 @@ import {
   useReplaceBookmarkAsset,
 } from "@hoarder/shared-react/hooks/bookmarks";
 import { getAssetUrl } from "@hoarder/shared-react/utils/assetUtils";
-import { ZAssetType, ZBookmark } from "@hoarder/shared/types/bookmarks";
+import {
+  BookmarkTypes,
+  ZAssetType,
+  ZBookmark,
+} from "@hoarder/shared/types/bookmarks";
 import {
   humanFriendlyNameForAssertType,
   isAllowedToAttachAsset,
@@ -94,10 +98,12 @@ export default function AttachmentBox({ bookmark }: { bookmark: ZBookmark }) {
     },
   });
 
-  if (!bookmark.assets.length) {
+  bookmark.assets.sort((a, b) => a.assetType.localeCompare(b.assetType));
+
+  if (bookmark.content.type == BookmarkTypes.ASSET) {
+    // Currently, we don't allow attaching assets to assets types.
     return null;
   }
-  bookmark.assets.sort((a, b) => a.assetType.localeCompare(b.assetType));
 
   return (
     <Collapsible>
-- 
cgit v1.2.3-70-g09d2

