This is an automated email from the ASF dual-hosted git repository.

juzhiyuan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git


The following commit(s) were added to refs/heads/master by this push:
     new 91a79a1  feat:  deleting hosts when hosts is empty (#503)
91a79a1 is described below

commit 91a79a1eaf2b6245d7f6cb7c0fe1f66080dd17a8
Author: litesun <7sunm...@gmail.com>
AuthorDate: Tue Sep 22 16:04:20 2020 +0800

    feat:  deleting hosts when hosts is empty (#503)
---
 src/pages/Route/List.tsx     | 2 +-
 src/pages/Route/transform.ts | 5 +++--
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/pages/Route/List.tsx b/src/pages/Route/List.tsx
index bad1132..92d7ac4 100644
--- a/src/pages/Route/List.tsx
+++ b/src/pages/Route/List.tsx
@@ -38,7 +38,7 @@ const Page: React.FC = () => {
       title: formatMessage({ id: 'route.list.domain.name' }),
       dataIndex: 'hosts',
       render: (_, record) =>
-        record.hosts.map((host) => (
+        (record.hosts || []).map((host) => (
           <Tag key={host} color="geekblue">
             {host}
           </Tag>
diff --git a/src/pages/Route/transform.ts b/src/pages/Route/transform.ts
index 6b07be3..3288b20 100644
--- a/src/pages/Route/transform.ts
+++ b/src/pages/Route/transform.ts
@@ -121,6 +121,7 @@ export const transformStepData = ({
       'redirectCode',
       'forceHttps',
       'redirectOption',
+      form1Data.hosts.filter(Boolean).length === 0 ? 'hosts' : '',
       form1Data.redirectOption === 'disabled' ? 'redirect' : '',
       form2Data.upstream_id ? 'upstream' : 'upstream_id',
     ]);
@@ -130,13 +131,13 @@ export const transformStepData = ({
     'name',
     'desc',
     'protocols',
-    'hosts',
     'uris',
     'methods',
     'redirect',
     'vars',
     'route_group_id',
     'route_group_name',
+    form1Data.hosts.filter(Boolean).length !== 0 ? 'hosts' : '',
   ]);
 };
 
@@ -191,7 +192,7 @@ export const transformRouteData = (data: RouteModule.Body) 
=> {
     status,
     protocols: protocols.filter((item) => item !== 'websocket'),
     websocket: protocols.includes('websocket'),
-    hosts,
+    hosts: (hosts || []).filter(Boolean).length === 0 ? [''] : hosts,
     paths: uris,
     methods,
   };

Reply via email to