fix: project path error (#7134) (#7136)

Co-authored-by: 青湛 <0x1304570@gmail.com>
apache · Mar 7, 2024 · 583912b · 583912b
1 parent 60b9677
commit 583912b
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 6 deletions.
diff --git a/config-ui/src/config/paths.ts b/config-ui/src/config/paths.ts
@@ -25,8 +25,22 @@ export const PATHS = {
   CONNECTIONS: () => `${PATH_PREFIX}/connections`,
   CONNECTION: (plugin: string, connectionId: ID) => `${PATH_PREFIX}/connections/${plugin}/${connectionId}`,
   PROJECTS: () => `${PATH_PREFIX}/projects`,
-  PROJECT: (pname: string, tab?: 'configuration' | 'status' | 'settings') =>
-    `${PATH_PREFIX}/projects/${encodeName(pname)}${tab ? `?tab=${tab}` : ''}`,
+  PROJECT: (
+    pname: string,
+    { tab, tabId }: { tab?: 'configuration' | 'status'; tabId?: 'blueprint' | 'webhook' | 'settings' } = {},
+  ) => {
+    let params = '';
+
+    if (tab && tabId) {
+      params = `?tab=${tab}&tabId=${tabId}`;
+    } else if (tab) {
+      params = `?tab=${tab}`;
+    } else if (tabId) {
+      params = `?tabId=${tabId}`;
+    }
+
+    return `${PATH_PREFIX}/projects/${encodeName(pname)}${params}`;
+  },
   PROJECT_CONNECTION: (pname: string, plugin: string, connectionId: ID) =>
     `${PATH_PREFIX}/projects/${encodeName(pname)}/${plugin}-${connectionId}`,
   BLUEPRINTS: () => `${PATH_PREFIX}/advanced/blueprints`,

diff --git a/config-ui/src/routes/blueprint/connection-detail/index.tsx b/config-ui/src/routes/blueprint/connection-detail/index.tsx
@@ -115,7 +115,7 @@ export const BlueprintConnectionDetailPage = () => {
 
     if (success) {
       handleShowTips();
-      navigate(pname ? PATHS.PROJECT(pname, 'configuration') : PATHS.BLUEPRINT(blueprint.id, 'configuration'));
+      navigate(pname ? PATHS.PROJECT(pname, { tab: 'configuration' }) : PATHS.BLUEPRINT(blueprint.id, 'configuration'));
     }
   };
 

diff --git a/config-ui/src/routes/project/detail/settings-panel.tsx b/config-ui/src/routes/project/detail/settings-panel.tsx
@@ -77,7 +77,7 @@ export const SettingsPanel = ({ project, onRefresh }: Props) => {
 
     if (success) {
       onRefresh();
-      navigate(PATHS.PROJECT(name, 'settings'));
+      navigate(PATHS.PROJECT(name, { tabId: 'settings' }));
     }
   };
 

diff --git a/config-ui/src/routes/project/home/index.tsx b/config-ui/src/routes/project/home/index.tsx
@@ -134,7 +134,7 @@ export const ProjectHomePage = () => {
             dataIndex: 'name',
             key: 'name',
             render: (name: string) => (
-              <Link to={PATHS.PROJECT(name, 'configuration')} style={{ color: '#292b3f' }}>
+              <Link to={PATHS.PROJECT(name, { tab: 'configuration' })} style={{ color: '#292b3f' }}>
                 {name}
               </Link>
             ),
@@ -192,7 +192,7 @@ export const ProjectHomePage = () => {
               <Button
                 type="primary"
                 icon={<SettingOutlined />}
-                onClick={() => navigate(PATHS.PROJECT(name, 'configuration'))}
+                onClick={() => navigate(PATHS.PROJECT(name, { tab: 'configuration' }))}
               />
             ),
           },