chore: add typings to localstorage interface

CSharperMantle · May 24, 2023 · cc2046b · cc2046b
1 parent 6b734e5
commit cc2046b
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 5 deletions.
diff --git a/src/customization/history/History.ts b/src/customization/history/History.ts
@@ -58,7 +58,7 @@ export class History implements ILocalStorageSerializable {
   }
 
   public static fromLocalStorage(): History {
-    const result = retrieve(HistoryLocalStorageKey)
+    const result = retrieve<History>(HistoryLocalStorageKey)
 
     if (isNil(result)) return History.Empty
 

diff --git a/src/customization/settings/Settings.ts b/src/customization/settings/Settings.ts
@@ -94,7 +94,7 @@ export class Settings implements ILocalStorageSerializable {
   }
 
   public static fromLocalStorage(): Settings {
-    const result = retrieve(SettingsLocalStorageKey)
+    const result = retrieve<Settings>(SettingsLocalStorageKey)
 
     if (isNil(result)) return Settings.Default
 

diff --git a/src/localstorage/LocalStorageManager.ts b/src/localstorage/LocalStorageManager.ts
@@ -25,18 +25,17 @@ export function store<T = unknown>(key: string, value: T): boolean {
   try {
     window.localStorage.setItem(key, JSON.stringify(value))
   } catch (err: unknown) {
-    console.warn(err)
     return false
   }
   return true
 }
 
-export function retrieve(key: string): unknown {
+export function retrieve<T = unknown>(key: string): T | null {
   if (!isBrowser) return null
 
   const result = window.localStorage.getItem(key)
   if (!isNil(result)) {
-    return JSON.parse(result)
+    return JSON.parse(result) as T
   } else {
     return null
   }