fix(settings): 🐛 Fixed the issue where you couldn't save the Plex set…

…tings
anojht · Oct 8, 2021 · 89a856a · 89a856a
1 parent 1ef45dc
commit 89a856a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/Ombi/.vscode/settings.json b/src/Ombi/.vscode/settings.json
@@ -14,6 +14,7 @@
     "conventionalCommits.scopes": [
         "discover",
         "request-limits",
-        "notifications"
+        "notifications",
+        "settings"
     ]
 }
diff --git a/src/Ombi/ClientApp/src/app/settings/plex/plex.component.ts b/src/Ombi/ClientApp/src/app/settings/plex/plex.component.ts
@@ -63,6 +63,7 @@ export class PlexComponent implements OnInit, OnDestroy {
         server.plexAuthToken = selectedServer.accessToken;
         server.port = parseInt(selectedServer.port);
         server.ssl = selectedServer.scheme === "http" ? false : true;
+        server.serverHostname = "";
 
         this.notificationService.success(`Selected ${server.name}!`);
     }
@@ -128,7 +129,7 @@ export class PlexComponent implements OnInit, OnDestroy {
         let invalid = false;
 
         this.settings.servers.forEach(server => {
-            if (server.serverHostname.length > 0 && !server.serverHostname.startsWith("http")) {
+            if (server.serverHostname && server.serverHostname.length > 0 && !server.serverHostname.startsWith("http")) {
                 invalid = true;
             }
         });