fix(alerting): fixed error handling

When deleting an application that didn't exist, no error was
returned.
Now the error is correctly visualised to the user together
with the details of what happened.
This required the admin client
library to be updated.

package.json

@@ -3,7 +3,7 @@
   "version": "0.1.0",
   "private": true,
   "dependencies": {
-    "@aerogear/unifiedpush-admin-client": "4.1.0",
+    "@aerogear/unifiedpush-admin-client": "4.2.1",
     "@fortawesome/fontawesome-free": "^5.13.0",
     "@patternfly/react-core": "4.18.5",
     "@types/react-router-dom": "^5.1.5",

src/application/ApplicationDetail/panels/dialogs/RenewVariantSecret.tsx

@@ -55,13 +55,11 @@ export class RenewVariantSecret extends Component<Props, State> {
           this.props.variant.variantID
         )
         .execute();
-      console.log({ old: this.props.variant, new: variant });
       await this.setState({ refreshing: false });
       this.props.onRefreshed(variant);
     };
 
     const handleAppNameInput = (value: string) => {
-      console.log('verify', value);
       if (value === this.props.variant.name) {
         this.setState({ nameValid: ValidatedOptions.success });
       } else {

src/application/ApplicationList/ApplicationList.tsx

@@ -142,13 +142,11 @@ export class ApplicationList extends Component<Props, State> {
                 variant={PaginationVariant.bottom}
                 onNextClick={(_event, currentPage) => {
                   this.setState({ currentPage });
-                  console.log(currentPage);
                   refresh(currentPage - 1);
                 }}
                 onPreviousClick={(_event, currentPage) => {
                   this.setState({ currentPage });
                   refresh(currentPage - 1);
-                  console.log(currentPage);
                 }}
               />
             </>

src/application/VariantForms/VariantSelectionForm.tsx

@@ -135,7 +135,6 @@ export class VariantSelectionForm extends Component<Props, State> {
                 variant: VariantDefinition,
                 variantType: VariantType
               ) => {
-                console.log('Creating Variant');
                 try {
                   const newVariant = await UpsClientFactory.getUpsClient()
                     .variants[variantType].create(

src/application/crud/CreateApplicationPage.tsx

@@ -15,6 +15,10 @@ import {
 } from '@patternfly/react-core';
 import { UpsClientFactory } from '../../utils/UpsClientFactory';
 import { PushApplication } from '@aerogear/unifiedpush-admin-client';
+import {
+  ApplicationListContext,
+  ContextInterface,
+} from '../../context/Context';
 
 interface State {
   appName: string;
@@ -39,7 +43,7 @@ export class CreateApplicationPage extends Component<Props, State> {
         .execute();
       this.props.onFinished(app);
     } catch (err) {
-      console.log(err);
+      (this.context as ContextInterface).alert(err);
     }
   };
 
@@ -88,3 +92,4 @@ export class CreateApplicationPage extends Component<Props, State> {
     );
   }
 }
+CreateApplicationPage.contextType = ApplicationListContext;

src/application/crud/DeleteApplicationPage.tsx

@@ -8,6 +8,10 @@ import {
   Modal,
 } from '@patternfly/react-core';
 import { PushApplication } from '@aerogear/unifiedpush-admin-client';
+import {
+  ApplicationListContext,
+  ContextInterface,
+} from '../../context/Context';
 
 interface State {
   name: string;
@@ -40,7 +44,7 @@ export class DeleteApplicationPage extends Component<Props, State> {
         this.props.close();
       }
     } catch (err) {
-      console.log(err);
+      (this.context as ContextInterface).alert(err);
     }
   };
 
@@ -66,6 +70,7 @@ export class DeleteApplicationPage extends Component<Props, State> {
             fieldId="simple-form-input"
           >
             <TextInput
+              id="app-name"
               className="formInput"
               onChange={value => this.setState({ name: value })}
               isRequired
@@ -91,3 +96,4 @@ export class DeleteApplicationPage extends Component<Props, State> {
     );
   }
 }
+DeleteApplicationPage.contextType = ApplicationListContext;

src/application/crud/UpdateApplicationPage.tsx

@@ -8,6 +8,10 @@ import {
   Modal,
 } from '@patternfly/react-core';
 import { PushApplication } from '@aerogear/unifiedpush-admin-client';
+import {
+  ApplicationListContext,
+  ContextInterface,
+} from '../../context/Context';
 
 interface State {
   name?: string;
@@ -35,7 +39,7 @@ export class UpdateApplicationPage extends Component<Props, State> {
         .execute();
       this.props.close();
     } catch (err) {
-      console.log(err);
+      (this.context as ContextInterface).alert(err);
     }
   };
 
@@ -75,3 +79,4 @@ export class UpdateApplicationPage extends Component<Props, State> {
     );
   }
 }
+UpdateApplicationPage.contextType = ApplicationListContext;