diff --git a/src/components/people/PersonMergeDropdown.tsx b/src/components/people/PersonMergeDropdown.tsx index 710d8e2..4ae42ba 100644 --- a/src/components/people/PersonMergeDropdown.tsx +++ b/src/components/people/PersonMergeDropdown.tsx @@ -18,8 +18,6 @@ import { searchPeople, } from "@/handlers/api/people.handler"; import { IPerson } from "@/types/person"; -import { set } from "date-fns"; -import { CommandLoading } from "cmdk"; import { Avatar } from "../ui/avatar"; import { useEffect, useMemo, useRef, useState } from "react"; import { useToast } from "../ui/use-toast"; @@ -31,7 +29,7 @@ import { DropdownMenuItem, DropdownMenuTrigger, } from "../ui/dropdown-menu"; -import { CaretDownIcon, FaceIcon } from "@radix-ui/react-icons"; +import { CaretDownIcon } from "@radix-ui/react-icons"; import FaceThumbnail from "./merge/FaceThumbnail"; import ErrorBlock from "../shared/ErrorBlock"; @@ -103,6 +101,7 @@ export function PersonMergeDropdown({ const isAlreadySelected = selectedPeople.some((p) => p.id === value.id); if (isAlreadySelected) { setSelectedPeople(selectedPeople.filter((p) => p.id !== value.id)); + return; } if (selectedPeople.length >= 5) { toast({ @@ -168,7 +167,7 @@ export function PersonMergeDropdown({ useEffect(() => { if (open && !similarPeople.length) fetchSuggestions(); - }, [open, person.id]); + }, [open, person.id, similarPeople]); const renderPeopleList = (people: IPerson[], title: string) => { return ( @@ -286,7 +285,7 @@ export function PersonMergeDropdown({ {selectedPeople.map((person) => (
Untagged person}