From 7d163f2189c6f8080c0a9185cacab52b1b2cd5c0 Mon Sep 17 00:00:00 2001 From: MohamedBassem Date: Fri, 26 Apr 2024 12:38:02 +0100 Subject: feature: Allow users to delete all unused tags in one go --- apps/web/components/dashboard/tags/AllTagsView.tsx | 54 ++++++++++++++++++++-- packages/shared-react/hooks/tags.ts | 14 ++++++ packages/trpc/routers/tags.ts | 26 ++++++++++- 3 files changed, 89 insertions(+), 5 deletions(-) diff --git a/apps/web/components/dashboard/tags/AllTagsView.tsx b/apps/web/components/dashboard/tags/AllTagsView.tsx index ce780a2f..a16dd759 100644 --- a/apps/web/components/dashboard/tags/AllTagsView.tsx +++ b/apps/web/components/dashboard/tags/AllTagsView.tsx @@ -1,6 +1,8 @@ "use client"; import Link from "next/link"; +import { ActionButton } from "@/components/ui/action-button"; +import ActionConfirmingDialog from "@/components/ui/action-confirming-dialog"; import { Button } from "@/components/ui/button"; import { Collapsible, @@ -9,13 +11,50 @@ import { } from "@/components/ui/collapsible"; import InfoTooltip from "@/components/ui/info-tooltip"; import { Separator } from "@/components/ui/separator"; +import { toast } from "@/components/ui/use-toast"; import { api } from "@/lib/trpc"; import { X } from "lucide-react"; import type { ZGetTagResponse } from "@hoarder/shared/types/tags"; +import { useDeleteUnusedTags } from "@hoarder/shared-react/hooks/tags"; import DeleteTagConfirmationDialog from "./DeleteTagConfirmationDialog"; +function DeleteAllUnusedTags({ numUnusedTags }: { numUnusedTags: number }) { + const { mutate, isPending } = useDeleteUnusedTags({ + onSuccess: () => { + toast({ + description: `Deleted all ${numUnusedTags} unused tags`, + }); + }, + onError: () => { + toast({ + description: "Something went wrong", + variant: "destructive", + }); + }, + }); + return ( + ( + mutate()} + > + DELETE THEM ALL + + )} + > + + + ); +} + function TagPill({ id, name, @@ -102,9 +141,18 @@ export default function AllTagsView({ - - - +
+ + + + {emptyTags.length > 0 && ( + + )} +
{tagsToPill(emptyTags)}
diff --git a/packages/shared-react/hooks/tags.ts b/packages/shared-react/hooks/tags.ts index d3129fed..6ce0f0c9 100644 --- a/packages/shared-react/hooks/tags.ts +++ b/packages/shared-react/hooks/tags.ts @@ -53,3 +53,17 @@ export function useDeleteTag( }, }); } + +export function useDeleteUnusedTags( + ...opts: Parameters +) { + const apiUtils = api.useUtils(); + + return api.tags.deleteUnused.useMutation({ + ...opts[0], + onSuccess: (res, req, meta) => { + apiUtils.tags.list.invalidate(); + return opts[0]?.onSuccess?.(res, req, meta); + }, + }); +} diff --git a/packages/trpc/routers/tags.ts b/packages/trpc/routers/tags.ts index b95570ae..dc70b068 100644 --- a/packages/trpc/routers/tags.ts +++ b/packages/trpc/routers/tags.ts @@ -1,5 +1,5 @@ import { experimental_trpcMiddleware, TRPCError } from "@trpc/server"; -import { and, eq, inArray } from "drizzle-orm"; +import { and, eq, inArray, notExists } from "drizzle-orm"; import { z } from "zod"; import type { ZAttachedByEnum } from "@hoarder/shared/types/tags"; @@ -115,6 +115,28 @@ export const tagsAppRouter = router({ } // TODO: Update affected bookmarks in search index }), + deleteUnused: authedProcedure + .output( + z.object({ + deletedTags: z.number(), + }), + ) + .mutation(async ({ ctx }) => { + const res = await ctx.db + .delete(bookmarkTags) + .where( + and( + eq(bookmarkTags.userId, ctx.user.id), + notExists( + ctx.db + .select({ id: tagsOnBookmarks.tagId }) + .from(tagsOnBookmarks) + .where(eq(tagsOnBookmarks.tagId, bookmarkTags.id)), + ), + ), + ); + return { deletedTags: res.changes }; + }), update: authedProcedure .input( z.object({ @@ -261,7 +283,7 @@ export const tagsAppRouter = router({ tagId: input.intoTagId, })), ) - .onConflictDoNothing() + .onConflictDoNothing(); } // Delete the old tags -- cgit v1.2.3-70-g09d2