Merge pull request #49 from ww24/fix/restore-default-settings

fix(settings): fix restore default settings
ww24 · Jun 30, 2024 · 2efe658 · 2efe658
2 parents 94c59b5 + 338acdf
commit 2efe658
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 1 deletion.
diff --git a/src/configuration.ts b/src/configuration.ts
@@ -33,6 +33,10 @@ export class Configuration {
         this.updatedAt = 0
         this.userId = ''
     }
+    static restoreDefault({ userId }: Configuration): Configuration {
+        const config = new Configuration()
+        return { ...config, userId }
+    }
     static videoFormat(config: Configuration): VideoFormat {
         if (config.videoFormat.videoBitrate === 0) {
             return {

diff --git a/src/element/settings.ts b/src/element/settings.ts
@@ -189,7 +189,7 @@ export class Settings extends LitElement {
     }
     private async restore() {
         const oldVal = this.config
-        this.config = new Configuration()
+        this.config = Configuration.restoreDefault(this.config)
         this.requestUpdate('config', oldVal)
         this.resetValidityError()
         Settings.setConfiguration(this.config)

diff --git a/src/service_worker.ts b/src/service_worker.ts
@@ -19,6 +19,7 @@ chrome.runtime.onInstalled.addListener(async () => {
     if (config.userId === '') {
         config.userId = uuidv7()
     }
+    await storage.set(Configuration.key, config)
     console.debug('config:', config)
 
     const msg: OffscreenSyncConfigMessage = {