Merge pull request alibaba#951 from loadchange/hotfix_service_api

Fix: delete service add group name
LuanLouis · Mar 22, 2019 · 7ce7681 · 7ce7681
2 parents bf162f7 + 8ff2ced
commit 7ce7681
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/...c/main/resources/static/console-fe/src/pages/ServiceManagement/ServiceList/ServiceList.js b/...c/main/resources/static/console-fe/src/pages/ServiceManagement/ServiceList/ServiceList.js
@@ -104,7 +104,7 @@ class ServiceList extends React.Component {
     setTimeout(() => this.queryServiceList());
   };
 
-  deleteService(serviceName) {
+  deleteService(service) {
     const { locale = {} } = this.props;
     const { prompt, promptDelete } = locale;
     Dialog.confirm({
@@ -113,7 +113,7 @@ class ServiceList extends React.Component {
       onOk: () => {
         request({
           method: 'DELETE',
-          url: `v1/ns/service?serviceName=${serviceName}`,
+          url: `v1/ns/service?serviceName=${service.name}&groupName=${service.groupName}`,
           dataType: 'text',
           beforeSend: () => this.openLoading(),
           success: res => {
@@ -241,7 +241,7 @@ class ServiceList extends React.Component {
                       <Button
                         style={{ marginLeft: 12 }}
                         type="normal"
-                        onClick={() => this.deleteService(record.name)}
+                        onClick={() => this.deleteService(record)}
                       >
                         {deleteAction}
                       </Button>

diff --git a/console/src/main/resources/static/js/main.js b/console/src/main/resources/static/js/main.js