Skip to content

Commit 56928d2

Browse files
committed
Move first asset selection logic to reducer
1 parent 8baca9a commit 56928d2

File tree

2 files changed

+4
-6
lines changed

2 files changed

+4
-6
lines changed

web/src/components/devtools/TeamMemory.tsx

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -18,12 +18,6 @@ export const TeamMemory: React.FC = () => {
1818
const assetsLoading = useSelector((state: RootState) => state.settings.assetsLoading)
1919
const useTeamMemory = useSelector((state: RootState) => state.settings.useTeamMemory)
2020

21-
// Auto-select first asset when assets are loaded and none is selected
22-
React.useEffect(() => {
23-
if (availableAssets.length > 0 && !selectedAssetId) {
24-
dispatch(setSelectedAssetId(availableAssets[0].slug))
25-
}
26-
}, [availableAssets, selectedAssetId])
2721

2822
const handleAssetSelection = (assetSlug: string) => {
2923
dispatch(setSelectedAssetId(assetSlug === '' ? null : assetSlug))

web/src/state/settings/reducer.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -304,6 +304,10 @@ export const settingsSlice = createSlice({
304304
},
305305
setAvailableAssets: (state, action: PayloadAction<AssetInfo[]>) => {
306306
state.availableAssets = action.payload
307+
// Auto-select first asset if no asset is currently selected and assets are available
308+
if (action.payload.length > 0 && !state.selectedAssetId) {
309+
state.selectedAssetId = action.payload[0].slug
310+
}
307311
},
308312
setSelectedAssetId: (state, action: PayloadAction<string | null>) => {
309313
state.selectedAssetId = action.payload

0 commit comments

Comments
 (0)