From: Lott, Christopher (cl778h) <cl778h@att.com>
Date: Wed, 21 Aug 2019 18:31:56 +0000 (-0400)
Subject: Modify FE method to accept response code 204
X-Git-Tag: R2~33
X-Git-Url: https://gerrit.o-ran-sc.org/r/gitweb?a=commitdiff_plain;h=02a955a2c99d6e6c58e72987a8b3f0a75e037342;p=portal%2Fric-dashboard.git

Modify FE method to accept response code 204

Change-Id: Ie93005d683df15a3c345c541de93533accd2ccb5
Signed-off-by: Lott, Christopher (cl778h) <cl778h@att.com>
---

diff --git a/docs/release-notes.rst b/docs/release-notes.rst
index 1eb16cb8..97e59ec5 100644
--- a/docs/release-notes.rst
+++ b/docs/release-notes.rst
@@ -23,7 +23,7 @@ Version 1.2.1, ? Aug 2019
 -------------------------
 * Add EPSDK-FW user management and Portal security
 
-Version 1.2.0, 20 Aug 2019
+Version 1.2.0, 21 Aug 2019
 --------------------------
 * Split URL properties into prefix/suffix parts
 * Add jacoco plugin to back-end for code coverage
@@ -42,7 +42,7 @@ Version 1.2.0, 20 Aug 2019
 * Synch A1 method paths in front-end and back-end
 * Add xapp dynamic configuration feature
 * Disable x-frame-options response header
-* Repair app manager undeploy-app method
+* Repair app manager undeploy-app back/front methods
 * Display AC xAPP metrics data via Kibana source (metrics.url.ac) on dashboard
 
 Version 1.0.5, 5 July 2019
diff --git a/webapp-frontend/src/app/app-control/app-control.component.ts b/webapp-frontend/src/app/app-control/app-control.component.ts
index 6ef4b161..057b9761 100644
--- a/webapp-frontend/src/app/app-control/app-control.component.ts
+++ b/webapp-frontend/src/app/app-control/app-control.component.ts
@@ -75,14 +75,9 @@ export class AppControlComponent implements OnInit {
         if (res) {
           this.appMgrSvc.undeployXapp(app.xapp).subscribe(
             ( httpResponse: HttpResponse<Object>) => {
+              // Answers 204/No content on success
+              this.notificationService.success('App undeployed successfully!');
               this.dataSource.loadTable();
-              switch (httpResponse.status) {
-                case 200:
-                  this.notificationService.success('App undeployed successfully!');
-                  break;
-                default:
-                  this.notificationService.warn('App undeploy failed.');
-              }
             },
             ( (her: HttpErrorResponse) => {
               // the error field should have an ErrorTransport object