From 688cf3786cde1d4030dfb12e7d550677c7ab6735 Mon Sep 17 00:00:00 2001 From: Don McKenzie Date: Tue, 12 Nov 2024 11:21:25 -0500 Subject: [PATCH] add python, groovy and clarifiy OS is UA reported --- packages/code-studio/src/settings/SettingsUtils.tsx | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/packages/code-studio/src/settings/SettingsUtils.tsx b/packages/code-studio/src/settings/SettingsUtils.tsx index a05da35448..f4ef082c45 100644 --- a/packages/code-studio/src/settings/SettingsUtils.tsx +++ b/packages/code-studio/src/settings/SettingsUtils.tsx @@ -49,13 +49,17 @@ export function getFormattedVersionInfo( Number(parseFloat(ua.browser.version ?? '')) || '' }`; const os = `${ua.os.name ?? ''} ${ua.os.version ?? ''}`; + const pythonVersion = serverConfigValues.get('python.version') ?? ''; return { 'Engine Version': serverConfigValues.get('deephaven.version') ?? 'Unknown', 'Web UI Version': import.meta.env.npm_package_version ?? 'Unknown', + // Python version is only available in python sessions + ...(pythonVersion !== '' ? { 'Python Version': pythonVersion } : {}), 'Java Version': serverConfigValues.get('java.version') ?? 'Unknown', + 'Groovy Version': serverConfigValues.get('groovy.version') ?? 'Unknown', 'Barrage Version': serverConfigValues.get('barrage.version') ?? 'Unknown', 'Browser Name': browser.trim() || 'Unknown', - 'OS Name': os.trim() || 'Unknown', + 'User Agent OS': os.trim() || 'Unknown', }; }