From 8a1309536b76ba86872ca2e78aa695d9fd80c8cc Mon Sep 17 00:00:00 2001 From: MohamedBassem 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 ( -- cgit v1.2.3-70-g09d2