chore: add loading in route page (#2287)

apache · Jan 25, 2022 · 4c8972e · 4c8972e
1 parent 131601d
commit 4c8972e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/web/cypress/integration/route/create-edit-duplicate-delete-route.spec.js b/web/cypress/integration/route/create-edit-duplicate-delete-route.spec.js
@@ -315,6 +315,9 @@ context('Create and Delete Route', () => {
         .within(() => {
           cy.contains('OK').click();
         });
+      cy.get(selector.deleteAlert).within(()=>{
+        cy.get('.ant-btn-loading-icon').should('be.visible');
+      })
       cy.get(selector.notification).should('contain', data.deleteRouteSuccess);
       cy.get(selector.notificationCloseIcon).click();
     });

diff --git a/web/src/pages/Route/List.tsx b/web/src/pages/Route/List.tsx
@@ -264,7 +264,7 @@ const Page: React.FC = () => {
               </>
             ),
             onOk: () => {
-              remove(record.id!).then(() => {
+              return remove(record.id!).then(() => {
                 handleTableActionSuccessResponse(
                   `${formatMessage({ id: 'component.global.delete' })} ${formatMessage({
                     id: 'menu.routes',