fix: 🐛 fixed variant doc links in the wizard

The wizard was always pointing to android docs
aerogear · Sep 16, 2020 · 4e55993 · 4e55993
1 parent 37db56c
commit 4e55993
Show file tree

Hide file tree

Showing 2 changed files with 42 additions and 44 deletions.
diff --git a/src/application/ApplicationDetail/SenderAPI.tsx b/src/application/ApplicationDetail/SenderAPI.tsx
@@ -117,7 +117,7 @@ export class SenderAPI extends Component<Props, State> {
             <Text component={'small'}>
               First add{' '}
               <code className={'code'}>unifiedpush-java-client.jar</code> as a{' '}
-              <a href={'links.pushApplications.senderAPI.docs.java_client'}>
+              <a href={getLink('sender-api-java')}>
                 dependency to your Java project
               </a>
               .
@@ -149,7 +149,7 @@ export class SenderAPI extends Component<Props, State> {
           <Tab eventKey={'node-sender-api'} title="Node.js Sender API">
             <Title headingLevel={'h1'}>Set up Node.js Sender API</Title>
             <Text component={'small'}>
-              First add <code className={'code'}>unifiedpush-jnode-sender</code>{' '}
+              First add <code className={'code'}>unifiedpush-node-sender</code>{' '}
               as a{' '}
               <a href={getLink('sender-api-nodejs')}>
                 dependency to your project

diff --git a/src/application/wizard/SetupPage.tsx b/src/application/wizard/SetupPage.tsx
@@ -1,4 +1,4 @@
-import React, { Component, ReactNode } from 'react';
+import React, { Component } from 'react';
 import {
   TextContent,
   Text,
@@ -37,6 +37,41 @@ export class SetupPage extends Component<Props> {
     const context = this.context as ContextInterface;
     const getLink = (key: string) => _getLink(context.upsConfig, key);
 
+    const getCodeSnippet = () => {
+      switch (context.selectedVariant?.type) {
+        case 'android':
+          return (
+            <AndroidCodeSnippets
+              app={this.props.app}
+              variant={context.selectedVariant! as AndroidVariant}
+            />
+          );
+        case 'web_push':
+          return (
+            <WebPushCodeSnippets
+              app={this.props.app}
+              variant={context.selectedVariant! as WebPushVariant}
+            />
+          );
+        case 'ios':
+          return (
+            <IOSCertCodeSnippets
+              app={this.props.app}
+              variant={context.selectedVariant! as IOSVariant}
+            />
+          );
+        case 'ios_token':
+          return (
+            <IOSTokenCodeSnippets
+              app={this.props.app}
+              variant={context.selectedVariant! as IOSTokenVariant}
+            />
+          );
+        default:
+          return <></>;
+      }
+    };
+
     return (
       <>
         <Page>
@@ -68,52 +103,15 @@ export class SetupPage extends Component<Props> {
               <TextListItem>Click Next(below)</TextListItem>
             </TextList>
           </TextContent>
-          <ApplicationListContext.Consumer>
-            {({ selectedVariant }: ContextInterface): ReactNode => {
-              if (selectedVariant?.type === 'android') {
-                return (
-                  <AndroidCodeSnippets
-                    app={this.props.app}
-                    variant={selectedVariant! as AndroidVariant}
-                  />
-                );
-              }
-              if (selectedVariant?.type === 'web_push') {
-                return (
-                  <WebPushCodeSnippets
-                    app={this.props.app}
-                    variant={selectedVariant! as WebPushVariant}
-                  />
-                );
-              }
-              if (selectedVariant?.type === 'ios') {
-                return (
-                  <IOSCertCodeSnippets
-                    app={this.props.app}
-                    variant={selectedVariant! as IOSVariant}
-                  />
-                );
-              }
-              if (selectedVariant?.type === 'ios_token') {
-                return (
-                  <IOSTokenCodeSnippets
-                    app={this.props.app}
-                    variant={selectedVariant! as IOSTokenVariant}
-                  />
-                );
-              }
-
-              return () => {
-                console.log('missing code snippets');
-              };
-            }}
-          </ApplicationListContext.Consumer>
+          {getCodeSnippet()}
           <TextContent>
             <Text component={TextVariants.p}>
               Next we are going to send a test notification. Make sure you {''}
               <Text
                 component={TextVariants.a}
-                href={getLink('build-and-deploy-android')}
+                href={getLink(
+                  `build-and-deploy-${context.selectedVariant?.type}`
+                )}
               >
                 build or deploy your app
               </Text>