fix: prevent orphaned thread record when deleting active conversation (…

…#1883) * fix: prevent orphaned thread record when deleting active conversation * refactor: optimize thread deletion navigation logic
Chainlit · Feb 11, 2025 · 0fb811b · 0fb811b
1 parent 9baa6e0
commit 0fb811b
Showing 1 changed file with 7 additions and 9 deletions.
diff --git a/frontend/src/components/LeftSidebar/ThreadList.tsx b/frontend/src/components/LeftSidebar/ThreadList.tsx
@@ -101,25 +101,23 @@ export function ThreadList({
 
   const handleDeleteThread = () => {
     if (!threadIdToDelete) return;
+    if (
+      threadIdToDelete === idToResume ||
+      threadIdToDelete === currentThreadId
+    ) {
+      clear();
+    }
 
     toast.promise(apiClient.deleteThread(threadIdToDelete), {
       loading: (
         <Translator path="threadHistory.thread.actions.delete.inProgress" />
       ),
       success: () => {
-        if (
-          threadIdToDelete === idToResume ||
-          threadIdToDelete === currentThreadId
-        ) {
-          clear();
-        }
-        if (threadIdToDelete === threadHistory.currentThreadId) {
-          navigate('/');
-        }
         setThreadHistory((prev) => ({
           ...prev,
           threads: prev?.threads?.filter((t) => t.id !== threadIdToDelete)
         }));
+        navigate('/');
         return (
           <Translator path="threadHistory.thread.actions.delete.success" />
         );