Merge pull request #3456 from janhq/fix/correct-time-weighted-retriev…

…al-toggle fix: correct time-weighted retrieval toggle
janhq · Aug 26, 2024 · 4228b9f · 4228b9f
2 parents 7fca45a + fe71380
commit 4228b9f
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 15 deletions.
diff --git a/web/app/search/layout.tsx b/web/app/search/layout.tsx
@@ -2,7 +2,7 @@
 
 import { useEffect } from 'react'
 
-import { AppConfiguration, getUserHomePath, joinPath } from '@janhq/core'
+import { AppConfiguration, getUserHomePath } from '@janhq/core'
 
 import { useSetAtom } from 'jotai'
 

diff --git a/web/containers/Providers/DataLoader.tsx b/web/containers/Providers/DataLoader.tsx
@@ -2,7 +2,7 @@
 
 import { Fragment, ReactNode, useEffect } from 'react'
 
-import { AppConfiguration, getUserHomePath, joinPath } from '@janhq/core'
+import { AppConfiguration, getUserHomePath } from '@janhq/core'
 import { useSetAtom } from 'jotai'
 
 import useAssistants from '@/hooks/useAssistants'

diff --git a/web/screens/Thread/ThreadRightPanel/Tools/index.tsx b/web/screens/Thread/ThreadRightPanel/Tools/index.tsx
@@ -183,19 +183,6 @@ const Tools = () => {
                           your specific use case."
                       />
                     </label>
-                    <div className="ml-auto flex items-center justify-between">
-                      <Switch
-                        name="use-time-weighted-retriever"
-                        className="mr-2"
-                        checked={
-                          activeThread?.assistants[0].tools[0]
-                            .useTimeWeightedRetriever || false
-                        }
-                        onChange={(e) =>
-                          onTimeWeightedRetrieverSwitchUpdate(e.target.checked)
-                        }
-                      />
-                    </div>
                   </div>
 
                   <div className="w-full">
@@ -223,6 +210,18 @@ const Tools = () => {
                                 also considers when they were added to give
                                 newer ones more importance."
                     />
+                    <div className="ml-auto flex items-center justify-between">
+                      <Switch
+                        name="use-time-weighted-retriever"
+                        checked={
+                          activeThread?.assistants[0].tools[0]
+                            .useTimeWeightedRetriever || false
+                        }
+                        onChange={(e) =>
+                          onTimeWeightedRetrieverSwitchUpdate(e.target.checked)
+                        }
+                      />
+                    </div>
                   </div>
                 </div>
                 <AssistantSetting