Remove History Turns config option and force value to 0
- Remove History Turns UI component from QuerySettings - Update settings store version to 17 with migration - Force history_turns parameter to always be 0 in queries - Prevent future modifications to history_turns setting
This commit is contained in:
parent
5b8989e4d9
commit
755a08edec
2 changed files with 14 additions and 39 deletions
|
|
@ -279,41 +279,6 @@ export default function QuerySettings() {
|
||||||
</div>
|
</div>
|
||||||
</>
|
</>
|
||||||
|
|
||||||
{/* History Turns */}
|
|
||||||
<>
|
|
||||||
<TooltipProvider>
|
|
||||||
<Tooltip>
|
|
||||||
<TooltipTrigger asChild>
|
|
||||||
<label htmlFor="history_turns" className="ml-1 cursor-help">
|
|
||||||
{t('retrievePanel.querySettings.historyTurns')}
|
|
||||||
</label>
|
|
||||||
</TooltipTrigger>
|
|
||||||
<TooltipContent side="left">
|
|
||||||
<p>{t('retrievePanel.querySettings.historyTurnsTooltip')}</p>
|
|
||||||
</TooltipContent>
|
|
||||||
</Tooltip>
|
|
||||||
</TooltipProvider>
|
|
||||||
<div>
|
|
||||||
<Input
|
|
||||||
id="history_turns"
|
|
||||||
type="number"
|
|
||||||
value={querySettings.history_turns ?? ''}
|
|
||||||
onChange={(e) => {
|
|
||||||
const value = e.target.value
|
|
||||||
handleChange('history_turns', value === '' ? '' : parseInt(value) || 0)
|
|
||||||
}}
|
|
||||||
onBlur={(e) => {
|
|
||||||
const value = e.target.value
|
|
||||||
if (value === '' || isNaN(parseInt(value))) {
|
|
||||||
handleChange('history_turns', 0)
|
|
||||||
}
|
|
||||||
}}
|
|
||||||
min={0}
|
|
||||||
placeholder={t('retrievePanel.querySettings.historyTurnsPlaceholder')}
|
|
||||||
className="h-9"
|
|
||||||
/>
|
|
||||||
</div>
|
|
||||||
</>
|
|
||||||
|
|
||||||
{/* User Prompt */}
|
{/* User Prompt */}
|
||||||
<>
|
<>
|
||||||
|
|
|
||||||
|
|
@ -185,10 +185,14 @@ const useSettingsStoreBase = create<SettingsState>()(
|
||||||
|
|
||||||
setRetrievalHistory: (history: Message[]) => set({ retrievalHistory: history }),
|
setRetrievalHistory: (history: Message[]) => set({ retrievalHistory: history }),
|
||||||
|
|
||||||
updateQuerySettings: (settings: Partial<QueryRequest>) =>
|
updateQuerySettings: (settings: Partial<QueryRequest>) => {
|
||||||
|
// Filter out history_turns to prevent changes, always keep it as 0
|
||||||
|
const filteredSettings = { ...settings }
|
||||||
|
delete filteredSettings.history_turns
|
||||||
set((state) => ({
|
set((state) => ({
|
||||||
querySettings: { ...state.querySettings, ...settings }
|
querySettings: { ...state.querySettings, ...filteredSettings, history_turns: 0 }
|
||||||
})),
|
}))
|
||||||
|
},
|
||||||
|
|
||||||
setShowFileName: (show: boolean) => set({ showFileName: show }),
|
setShowFileName: (show: boolean) => set({ showFileName: show }),
|
||||||
setShowLegend: (show: boolean) => set({ showLegend: show }),
|
setShowLegend: (show: boolean) => set({ showLegend: show }),
|
||||||
|
|
@ -197,7 +201,7 @@ const useSettingsStoreBase = create<SettingsState>()(
|
||||||
{
|
{
|
||||||
name: 'settings-storage',
|
name: 'settings-storage',
|
||||||
storage: createJSONStorage(() => localStorage),
|
storage: createJSONStorage(() => localStorage),
|
||||||
version: 16,
|
version: 17,
|
||||||
migrate: (state: any, version: number) => {
|
migrate: (state: any, version: number) => {
|
||||||
if (version < 2) {
|
if (version < 2) {
|
||||||
state.showEdgeLabel = false
|
state.showEdgeLabel = false
|
||||||
|
|
@ -284,6 +288,12 @@ const useSettingsStoreBase = create<SettingsState>()(
|
||||||
// Add documentsPageSize field for older versions
|
// Add documentsPageSize field for older versions
|
||||||
state.documentsPageSize = 10
|
state.documentsPageSize = 10
|
||||||
}
|
}
|
||||||
|
if (version < 17) {
|
||||||
|
// Force history_turns to 0 for all users
|
||||||
|
if (state.querySettings) {
|
||||||
|
state.querySettings.history_turns = 0
|
||||||
|
}
|
||||||
|
}
|
||||||
return state
|
return state
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Add table
Reference in a new issue