fix: Only display API version if present (#773)

* Only display API version if present * Warn if `info.version` is omitted, error if `info` is missing.
Redocly · Jan 17, 2019 · fb3cb36 · fb3cb36
1 parent c343db6
commit fb3cb36
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/src/components/ApiInfo/ApiInfo.tsx b/src/components/ApiInfo/ApiInfo.tsx
@@ -70,12 +70,18 @@ export class ApiInfo extends React.Component<ApiInfoProps> {
       )) ||
       null;
 
+    const version =
+      (info.version && (
+        <span>({info.version})</span>
+      )) ||
+      null;
+
     return (
       <Section>
         <Row>
           <MiddlePanel className="api-info">
             <ApiHeader>
-              {info.title} <span>({info.version})</span>
+              {info.title} {version}
             </ApiHeader>
             {!hideDownloadButton && (
               <p>

diff --git a/src/services/OpenAPIParser.ts b/src/services/OpenAPIParser.ts
@@ -63,6 +63,12 @@ export class OpenAPIParser {
     if (spec.openapi === undefined) {
       throw new Error('Document must be valid OpenAPI 3.0.0 definition');
     }
+    if (spec.info === undefined) {
+      throw new Error('OpenAPI 3.0.0 requires an `info` section');
+    }
+    if (spec.info.version === undefined) {
+      console.warn('OpenAPI 3.0.0 requires setting a `info.version` field, ignoring.')
+    }
   }
 
   preprocess(spec: OpenAPISpec) {