fix: collection sharing

Author: sailwebs

web/src/app/marketplace/collections/[collectionId]/collection-header.tsx

@@ -2,8 +2,6 @@
 
 import { SharedCollection } from '@/api';
 import { PageContent } from '@/components/page-container';
-import { useAppContext } from '@/components/providers/app-provider';
-import { Badge } from '@/components/ui/badge';
 import {
   Breadcrumb,
   BreadcrumbItem,
@@ -14,21 +12,17 @@ import {
 import { Button } from '@/components/ui/button';
 import {
   Card,
-  CardAction,
   CardDescription,
   CardHeader,
   CardTitle,
 } from '@/components/ui/card';
 import { Separator } from '@/components/ui/separator';
-import { apiClient } from '@/lib/api/client';
 import { cn } from '@/lib/utils';
 import _ from 'lodash';
-import { BookOpen, Files, Star, User, VectorSquare } from 'lucide-react';
+import { BookOpen, Files, VectorSquare } from 'lucide-react';
 import { useTranslations } from 'next-intl';
 import Link from 'next/link';
-import { usePathname, useRouter } from 'next/navigation';
-import { useCallback, useMemo } from 'react';
-import { FaStar } from 'react-icons/fa6';
+import { usePathname } from 'next/navigation';
 
 export const CollectionHeader = ({
   className,
@@ -37,47 +31,13 @@ export const CollectionHeader = ({
   className?: string;
   collection: SharedCollection;
 }) => {
-  const router = useRouter();
   const pathname = usePathname();
 
-  const { user, signIn } = useAppContext();
-  const page_collections = useTranslations('page_collections');
   const page_documents = useTranslations('page_documents');
   const page_marketplace = useTranslations('page_marketplace');
   const page_graph = useTranslations('page_graph');
   const sidebar_workspace = useTranslations('sidebar_workspace');
 
-  const isOwner = useMemo(
-    () => collection.owner_user_id === user?.id,
-    [collection.owner_user_id, user?.id],
-  );
-  const isSubscriber = useMemo(
-    () => collection.subscription_id !== null,
-    [collection.subscription_id],
-  );
-
-  const handleSubscribe = useCallback(async () => {
-    if (!user) {
-      signIn();
-      return;
-    }
-
-    if (isSubscriber) {
-      await apiClient.defaultApi.marketplaceCollectionsCollectionIdSubscribeDelete(
-        {
-          collectionId: collection.id,
-        },
-      );
-    } else {
-      await apiClient.defaultApi.marketplaceCollectionsCollectionIdSubscribePost(
-        {
-          collectionId: collection.id,
-        },
-      );
-    }
-    router.refresh();
-  }, [collection.id, isSubscriber, router, signIn, user]);
-
   return (
     <PageContent className={cn('flex flex-col gap-4 pb-0', className)}>
       <div className="flex items-center">
@@ -110,31 +70,6 @@ export const CollectionHeader = ({
       <Card className="gap-0 p-0">
         <CardHeader className="p-4 pb-0">
           <CardTitle className="mb-0 text-2xl">{collection.title}</CardTitle>
-          <CardAction className="text-muted-foreground flex flex-row items-center gap-4 text-xs">
-            {isOwner ? (
-              <Badge>{page_collections('mine')}</Badge>
-            ) : (
-              <div className="flex flex-row items-center gap-1">
-                <User className="size-4" />
-                <div className="max-w-60 truncate">
-                  {collection.owner_username}
-                </div>
-              </div>
-            )}
-            <Button
-              variant="outline"
-              size="sm"
-              hidden={isOwner}
-              onClick={handleSubscribe}
-              className="text-muted-foreground cursor-pointer text-xs"
-            >
-              {isSubscriber ? <FaStar className="text-orange-500" /> : <Star />}
-
-              {isSubscriber
-                ? page_collections('subscribed')
-                : page_collections('subscribe')}
-            </Button>
-          </CardAction>
         </CardHeader>
         <CardDescription className="mb-4 px-4">
           {_.truncate(collection.description || 'No description available', {

web/src/app/workspace/collections/[collectionId]/collection-sharing.tsx

@@ -47,12 +47,20 @@ export const CollectionSharing = ({
 
   const handleSharing = useCallback(async () => {
     if (!collection?.id) return;
-    await apiClient.defaultApi.collectionsCollectionIdSharingPost({
-      collectionId: collection.id,
-      collectionPublishRequest: {
-        group_ids: selectedDepartents,
-      },
-    });
+
+    if (selectedDepartents.length === 0) {
+      await apiClient.defaultApi.collectionsCollectionIdSharingDelete({
+        collectionId: collection.id,
+      });
+    } else {
+      await apiClient.defaultApi.collectionsCollectionIdSharingPost({
+        collectionId: collection.id,
+        collectionPublishRequest: {
+          group_ids: selectedDepartents,
+        },
+      });
+    }
+
     setSharingVisible(false);
     toast.success(common_tips('update_success'));
   }, [collection?.id, common_tips, selectedDepartents]);