From faf6fcd83e9457981aa92e2ae185cc92108c16fc Mon Sep 17 00:00:00 2001 From: AAfghahi <48933336+AAfghahi@users.noreply.github.com> Date: Fri, 9 Jul 2021 16:06:17 -0400 Subject: [PATCH] fix: Database List Sorted (#15619) * sorted the database list * revisions * cloned the array * one more time with feeling * added documentation link as well --- .../views/CRUD/data/database/DatabaseModal/ModalHeader.tsx | 7 ++++--- .../src/views/CRUD/data/database/DatabaseModal/index.tsx | 2 +- superset-frontend/src/views/CRUD/hooks.ts | 2 +- 3 files changed, 6 insertions(+), 5 deletions(-) diff --git a/superset-frontend/src/views/CRUD/data/database/DatabaseModal/ModalHeader.tsx b/superset-frontend/src/views/CRUD/data/database/DatabaseModal/ModalHeader.tsx index 3ca8542160..17457c07b5 100644 --- a/superset-frontend/src/views/CRUD/data/database/DatabaseModal/ModalHeader.tsx +++ b/superset-frontend/src/views/CRUD/data/database/DatabaseModal/ModalHeader.tsx @@ -27,11 +27,12 @@ import { } from './styles'; import { DatabaseForm, DatabaseObject } from '../types'; -export const DOCUMENTATION_LINK = - 'https://superset.apache.org/docs/databases/installing-database-drivers'; - const supersetTextDocs = getDatabaseDocumentationLinks(); +export const DOCUMENTATION_LINK = supersetTextDocs + ? supersetTextDocs.support + : 'https://superset.apache.org/docs/databases/installing-database-drivers'; + const irregularDocumentationLinks = { postgresql: 'https://superset.apache.org/docs/databases/postgres', mssql: 'https://superset.apache.org/docs/databases/sql-server', diff --git a/superset-frontend/src/views/CRUD/data/database/DatabaseModal/index.tsx b/superset-frontend/src/views/CRUD/data/database/DatabaseModal/index.tsx index b0e6ea51b7..662a178f30 100644 --- a/superset-frontend/src/views/CRUD/data/database/DatabaseModal/index.tsx +++ b/superset-frontend/src/views/CRUD/data/database/DatabaseModal/index.tsx @@ -559,7 +559,7 @@ const DatabaseModal: FunctionComponent = ({ onChange={setDatabaseModel} placeholder="Choose a database..." > - {availableDbs?.databases + {[...(availableDbs?.databases || [])] ?.sort((a: DatabaseForm, b: DatabaseForm) => a.name.localeCompare(b.name), ) diff --git a/superset-frontend/src/views/CRUD/hooks.ts b/superset-frontend/src/views/CRUD/hooks.ts index 57e5d02193..33c1fe353a 100644 --- a/superset-frontend/src/views/CRUD/hooks.ts +++ b/superset-frontend/src/views/CRUD/hooks.ts @@ -631,7 +631,7 @@ export function useAvailableDatabases() { const getAvailable = useCallback(() => { SupersetClient.get({ - endpoint: `/api/v1/database/available`, + endpoint: `/api/v1/database/available/`, }).then(({ json }) => { setAvailableDbs(json); });