From 373ba6a0357f7e92273aecce3e21df3f7c4dd633 Mon Sep 17 00:00:00 2001 From: Tim Roes Date: Mon, 28 Nov 2022 14:28:32 +0100 Subject: [PATCH] =?UTF-8?q?=F0=9F=AA=9F=20=F0=9F=A7=AA=20Make=20sure=20to?= =?UTF-8?q?=20filter=20out=20connectors=20correctly=20(#19732)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * Make sure to filter out connectors correctly * Rename variable --- airbyte-webapp/src/core/domain/connector/constants.ts | 2 +- .../SignupSourceDropdown/useGetSourceDefinitions.tsx | 5 +++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/airbyte-webapp/src/core/domain/connector/constants.ts b/airbyte-webapp/src/core/domain/connector/constants.ts index a8ca0d30e1877..11985fab16ce6 100644 --- a/airbyte-webapp/src/core/domain/connector/constants.ts +++ b/airbyte-webapp/src/core/domain/connector/constants.ts @@ -15,7 +15,7 @@ export const DEV_IMAGE_TAG = "dev"; * @param {string} workspaceId The workspace Id * @returns {array} List of connectorIds that should be filtered out */ -export const getExcludedConnectorIds = (workspaceId: string) => +export const getExcludedConnectorIds = (workspaceId?: string) => isCloudApp() ? [ "707456df-6f4f-4ced-b5c6-03f73bcad1c5", // hide Cassandra Destination https://github.com/airbytehq/airbyte-cloud/issues/2606 diff --git a/airbyte-webapp/src/packages/cloud/components/experiments/SignupSourceDropdown/useGetSourceDefinitions.tsx b/airbyte-webapp/src/packages/cloud/components/experiments/SignupSourceDropdown/useGetSourceDefinitions.tsx index a798487d61ce5..81f43ae1b8978 100644 --- a/airbyte-webapp/src/packages/cloud/components/experiments/SignupSourceDropdown/useGetSourceDefinitions.tsx +++ b/airbyte-webapp/src/packages/cloud/components/experiments/SignupSourceDropdown/useGetSourceDefinitions.tsx @@ -16,10 +16,11 @@ const fetchCatalog = async (): Promise => { }; export const useGetSourceDefinitions = () => { - return useQuery("cloud_catalog", fetchCatalog, { + return useQuery("cloud_catalog", fetchCatalog, { select: (data) => { + const excludedConnectorIds = getExcludedConnectorIds(); return data.sources - .filter(() => getExcludedConnectorIds("")) + .filter((source) => !excludedConnectorIds.includes(source.sourceDefinitionId)) .map((source) => { const icon = availableSourceDefinitions.sourceDefinitions.find( (src) => src.sourceDefinitionId === source.sourceDefinitionId