refactor: server.version must be present

joinmarket-webui · Sep 23, 2023 · 4d61119 · 4d61119
1 parent 34d1963
commit 4d61119
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 15 deletions.
diff --git a/src/constants/features.ts b/src/constants/features.ts
@@ -22,5 +22,5 @@ const __isFeatureEnabled = (name: Feature, version: SemVer): boolean => {
 }
 
 export const isFeatureEnabled = (name: Feature, serviceInfo: ServiceInfo): boolean => {
-  return !!serviceInfo.server?.version && __isFeatureEnabled(name, serviceInfo.server.version)
+  return !!serviceInfo.server && __isFeatureEnabled(name, serviceInfo.server.version)
 }
diff --git a/src/context/ServiceInfoContext.tsx b/src/context/ServiceInfoContext.tsx
@@ -81,7 +81,7 @@ type SessionInfo = {
 }
 type ServerInfo = {
   server?: {
-    version?: SemVer
+    version: SemVer
   }
 }
 
@@ -134,20 +134,12 @@ const ServiceInfoProvider = ({ children }: PropsWithChildren<{}>) => {
 
     Api.getGetinfo({ signal: abortCtrl.signal })
       .then((res) => (res.ok ? res.json() : Api.Helper.throwError(res)))
-      .then((data: JmGetInfoData) => {
-        dispatchServiceInfo({
-          server: {
-            version: toSemVer(data),
-          },
-        })
-      })
-      .catch((err) => {
-        const notFound = err.response.status === 404
-        if (notFound) {
+      .then((data: JmGetInfoData) => toSemVer(data))
+      .catch((_) => UNKNOWN_VERSION)
+      .then((version) => {
+        if (!abortCtrl.signal.aborted) {
           dispatchServiceInfo({
-            server: {
-              version: UNKNOWN_VERSION,
-            },
+            server: { version },
           })
         }
       })