Fix top level bookmark migration (#706)

This commit is contained in:
Chenlei Hu
2024-09-01 18:55:56 -04:00
committed by GitHub
parent 75e91137f0
commit d9a7537169
2 changed files with 5 additions and 3 deletions

View File

@@ -345,7 +345,9 @@ test.describe('Menu', () => {
test('Can migrate legacy bookmarks', async ({ comfyPage }) => {
await comfyPage.setSetting('Comfy.NodeLibrary.Bookmarks', [
'foo/',
'foo/KSampler (Advanced)'
'foo/KSampler (Advanced)',
'UNKNOWN',
'KSampler'
])
await comfyPage.setSetting('Comfy.NodeLibrary.Bookmarks.V2', [])
await comfyPage.page.reload()
@@ -355,7 +357,7 @@ test.describe('Menu', () => {
)
expect(
await comfyPage.getSetting('Comfy.NodeLibrary.Bookmarks.V2')
).toEqual(['foo/', 'foo/KSamplerAdvanced'])
).toEqual(['foo/', 'foo/KSamplerAdvanced', 'KSampler'])
})
})

View File

@@ -28,7 +28,7 @@ export const useNodeBookmarkStore = defineStore('nodeBookmark', () => {
const nodeDef = nodeDefStore.nodeDefsByDisplayName[displayName]
if (!nodeDef) return
addBookmark(`${category}/${nodeDef.name}`)
addBookmark(`${category === '' ? '' : category + '/'}${nodeDef.name}`)
})
settingStore.set('Comfy.NodeLibrary.Bookmarks', [])
}