Merge pull request #62 from pvdthings/fix-bookmarks-bug

Web - Fix bookmarks bug
pvdthings · Sep 9, 2024 · fd8ca7d · fd8ca7d
2 parents 9c7eaee + cd6c419
commit fd8ca7d
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/apps/web/package.json b/apps/web/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@library_os/web",
-	"version": "0.15.6",
+	"version": "0.15.7",
 	"private": true,
 	"scripts": {
 		"dev": "vite dev",

diff --git a/apps/web/src/lib/stores/bookmarks.ts b/apps/web/src/lib/stores/bookmarks.ts
@@ -5,9 +5,16 @@ import { derived, writable } from "svelte/store";
 const defaultValue = [];
 
 function createBookmarks() {
-  const initialValue = browser ? JSON.parse(window.localStorage.getItem('myList'))
+  let initialValue = browser ? JSON.parse(window.localStorage.getItem('myList'))
   ?? defaultValue : defaultValue;
 
+  // convert from old format to new
+  if (initialValue.length && (typeof initialValue[0]) !== 'string') {
+    console.log('Converting old bookmarks...');
+    initialValue = initialValue.map(b => b.id);
+    window.localStorage.setItem('myList', JSON.stringify(initialValue));
+  }
+
   const things = writable<ThingID[]>(initialValue);
 
   things.subscribe((value) => {