From 97378f1369338ac53b9ace3c1c2b053975f5402c Mon Sep 17 00:00:00 2001 From: giancarlo Date: Wed, 1 May 2024 14:33:32 +0700 Subject: [PATCH] Simplify MonitoringProvider component MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The MonitoringProvider component has been simplified. The “config” prop, including 'provider' and 'providerToken', has been removed as they are no longer needed. The implementation now directly uses the getMonitoringProvider function and removed the check for 'props.config'. Conditionally rendering the BaselimeProvider component has also been simplified. --- .../monitoring/api/src/components/provider.tsx | 17 ++--------------- .../baselime/src/monitoring-service.ts | 0 2 files changed, 2 insertions(+), 15 deletions(-) delete mode 100644 packages/monitoring/baselime/src/monitoring-service.ts diff --git a/packages/monitoring/api/src/components/provider.tsx b/packages/monitoring/api/src/components/provider.tsx index 1eb0135e2..266345cf6 100644 --- a/packages/monitoring/api/src/components/provider.tsx +++ b/packages/monitoring/api/src/components/provider.tsx @@ -21,26 +21,13 @@ const SentryProvider = lazy(async () => { }; }); -type Config = { - provider: InstrumentationProvider; - providerToken: string; -}; - -export function MonitoringProvider( - props: React.PropsWithChildren<{ config?: Config }>, -) { +export function MonitoringProvider(props: React.PropsWithChildren) { const provider = getMonitoringProvider(); - if (!props.config) { - return <>{props.children}; - } - switch (provider) { case InstrumentationProvider.Baselime: return ( - - {props.children} - + {props.children} ); case InstrumentationProvider.Sentry: diff --git a/packages/monitoring/baselime/src/monitoring-service.ts b/packages/monitoring/baselime/src/monitoring-service.ts deleted file mode 100644 index e69de29bb..000000000