Northern_Lights/cherry-studio
1
0
Fork 0
mirror of https://github.com/CherryHQ/cherry-studio.git synced 2025-12-26 03:31:24 +08:00
Code Issues Actions 3 Packages Projects Releases Wiki Activity

fix: use nullish coalescing for advanced property updates (#10921)

Browse Source 
Replaces logical OR with nullish coalescing when updating advanced server properties to allow empty string values, enabling users to clear fields instead of preserving previous values.
This commit is contained in:
Pleasure1234 2025-10-23 16:49:25 +01:00 committed by GitHub
parent d187adb0d3
commit 1ac746a40e
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/renderer/src/pages/settings/MCPSettings/McpSettings.tsx
View File

@ -275,11 +275,11 @@ const McpSettings: React.FC = () => {
searchKey: server.searchKey,
timeout: values.timeout || server.timeout,
longRunning: values.longRunning,
// Preserve existing advanced properties if not set in the form
provider: values.provider || server.provider,
providerUrl: values.providerUrl || server.providerUrl,
logoUrl: values.logoUrl || server.logoUrl,
tags: values.tags || server.tags
// Use nullish coalescing to allow empty strings (for deletion)
provider: values.provider ?? server.provider,
providerUrl: values.providerUrl ?? server.providerUrl,
logoUrl: values.logoUrl ?? server.logoUrl,
tags: values.tags ?? server.tags
}
// set stdio or sse server