From d1738aa7ddc24253e72b9a856125c93504afe6e3 Mon Sep 17 00:00:00 2001 From: Eric Duong Date: Fri, 15 Sep 2023 12:52:08 -0400 Subject: [PATCH] chore(data-warehouse): Make external tab the focus (#17469) * flip tabs * revert * revert --- frontend/src/scenes/appScenes.ts | 2 +- .../data-warehouse/DataWarehousePageTabs.tsx | 12 ++++++------ .../external/DataWarehouseExternalScene.tsx | 16 +++++++++------- 3 files changed, 16 insertions(+), 14 deletions(-) diff --git a/frontend/src/scenes/appScenes.ts b/frontend/src/scenes/appScenes.ts index b4ceb4a3967a7..4b6203b3aa294 100644 --- a/frontend/src/scenes/appScenes.ts +++ b/frontend/src/scenes/appScenes.ts @@ -38,7 +38,7 @@ export const appScenes: Record any> = { [Scene.EarlyAccessFeature]: () => import('./early-access-features/EarlyAccessFeature'), [Scene.Surveys]: () => import('./surveys/Surveys'), [Scene.Survey]: () => import('./surveys/Survey'), - [Scene.DataWarehouse]: () => import('./data-warehouse/posthog/DataWarehousePosthogScene'), + [Scene.DataWarehouse]: () => import('./data-warehouse/external/DataWarehouseExternalScene'), [Scene.DataWarehousePosthog]: () => import('./data-warehouse/posthog/DataWarehousePosthogScene'), [Scene.DataWarehouseExternal]: () => import('./data-warehouse/external/DataWarehouseExternalScene'), [Scene.DataWarehouseSavedQueries]: () => import('./data-warehouse/saved_queries/DataWarehouseSavedQueriesScene'), diff --git a/frontend/src/scenes/data-warehouse/DataWarehousePageTabs.tsx b/frontend/src/scenes/data-warehouse/DataWarehousePageTabs.tsx index f3ec71db47111..821746f7f59e6 100644 --- a/frontend/src/scenes/data-warehouse/DataWarehousePageTabs.tsx +++ b/frontend/src/scenes/data-warehouse/DataWarehousePageTabs.tsx @@ -13,8 +13,8 @@ export enum DataWarehouseTab { } const tabUrls = { - [DataWarehouseTab.Posthog]: urls.dataWarehousePosthog(), [DataWarehouseTab.External]: urls.dataWarehouseExternal(), + [DataWarehouseTab.Posthog]: urls.dataWarehousePosthog(), [DataWarehouseTab.Views]: urls.dataWarehouseSavedQueries(), } @@ -25,7 +25,7 @@ const dataWarehouseTabsLogic = kea({ }, reducers: { tab: [ - DataWarehouseTab.Posthog as DataWarehouseTab, + DataWarehouseTab.External as DataWarehouseTab, { setTab: (_, { tab }) => tab, }, @@ -58,14 +58,14 @@ export function DataWarehousePageTabs({ tab }: { tab: DataWarehouseTab }): JSX.E activeKey={tab} onChange={(t) => setTab(t)} tabs={[ - { - key: DataWarehouseTab.Posthog, - label: Posthog, - }, { key: DataWarehouseTab.External, label: External, }, + { + key: DataWarehouseTab.Posthog, + label: Posthog, + }, ...(featureFlags[FEATURE_FLAGS.DATA_WAREHOUSE_VIEWS] ? [ { diff --git a/frontend/src/scenes/data-warehouse/external/DataWarehouseExternalScene.tsx b/frontend/src/scenes/data-warehouse/external/DataWarehouseExternalScene.tsx index a4750db49090c..4b163af4135f4 100644 --- a/frontend/src/scenes/data-warehouse/external/DataWarehouseExternalScene.tsx +++ b/frontend/src/scenes/data-warehouse/external/DataWarehouseExternalScene.tsx @@ -30,13 +30,15 @@ export function DataWarehouseExternalScene(): JSX.Element { } buttons={ - - New Table - + !shouldShowProductIntroduction ? ( + + New Table + + ) : undefined } caption={