refactor unnecessary Id splitting and pass the document id to onSelect

adityaphasu · Sep 17, 2024 · c2e2864 · c2e2864
1 parent 454bca7
commit c2e2864
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/components/search-command.tsx b/components/search-command.tsx
@@ -44,8 +44,7 @@ export const SearchCommand = () => {
   }, [toggle]);
 
   const onSelect = (id: string) => {
-    const splitId = id.split("-")[0];
-    router.push(`/documents/${splitId}`);
+    router.push(`/documents/${id}`);
     onClose();
   };
 
@@ -64,7 +63,7 @@ export const SearchCommand = () => {
               key={document._id}
               value={`${document._id}-${document.title}`}
               title={document.title}
-              onSelect={onSelect}
+              onSelect={() => onSelect(document._id)}
             >
               {document.icon ? (
                 <p className="mr-2 text-[1.125rem]">{document.icon}</p>