From 5a6d90ae1b0bcb659dc5f54cfd7caf8a7d3d52dc Mon Sep 17 00:00:00 2001 From: Miguel Palau Zarza Date: Tue, 29 Jul 2025 14:32:15 -0600 Subject: [PATCH] refactor: use configure() for dynamic datasource extension URLs - Replace onClick handler with configure() function for dynamic path generation - Use context to build monitoring tool URL with datasource UID - Follows extension system design patterns for declarative configuration - Removes manual window.open() in favor of automatic path handling --- .../extensions/getDataSourceExtensionConfigs.tsx | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/public/app/features/datasources/extensions/getDataSourceExtensionConfigs.tsx b/public/app/features/datasources/extensions/getDataSourceExtensionConfigs.tsx index 1000d1fec8b..13996c249c0 100644 --- a/public/app/features/datasources/extensions/getDataSourceExtensionConfigs.tsx +++ b/public/app/features/datasources/extensions/getDataSourceExtensionConfigs.tsx @@ -26,10 +26,11 @@ export function getDataSourceExtensionConfigs(): PluginExtensionAddedLinkConfig[ if (context?.dataSource?.type !== 'prometheus') { return undefined; } - return {}; - }, - onClick: (_, { context }) => { - window.open(`https://monitoring-tool.com/datasource/${context!.dataSource.uid}`, '_blank'); + + // Return dynamic path with context + return { + path: `https://monitoring-tool.com/datasource/${context.dataSource.uid}`, + }; }, }),