Skip to content

Commit 2b3be11

Browse files
committed
fix(settings): eliminate double-parse in handlePaste by inlining handleKeyValuePaste
1 parent 9e09f22 commit 2b3be11

File tree

1 file changed

+5
-12
lines changed

1 file changed

+5
-12
lines changed

apps/sim/app/workspace/[workspaceId]/settings/components/credentials/credentials-manager.tsx

Lines changed: 5 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -892,17 +892,6 @@ export function CredentialsManager() {
892892
})
893893
}
894894

895-
const handleKeyValuePaste = (lines: string[]) => {
896-
const parsedVars = parseValidEnvVars(lines)
897-
if (parsedVars.length > 0) {
898-
setEnvVars((prev) => {
899-
const existingVars = prev.filter((v) => v.key || v.value)
900-
return [...existingVars, ...parsedVars, createEmptyEnvVar()]
901-
})
902-
scrollToBottom()
903-
}
904-
}
905-
906895
/**
907896
* Paste handler for personal env var rows.
908897
* Only prevents default when it actually handles the paste: KV patterns destructure into new rows,
@@ -932,7 +921,11 @@ export function CredentialsManager() {
932921
const parsedVars = parseValidEnvVars(lines)
933922
if (parsedVars.length > 0) {
934923
e.preventDefault()
935-
handleKeyValuePaste(lines)
924+
setEnvVars((prev) => {
925+
const existingVars = prev.filter((v) => v.key || v.value)
926+
return [...existingVars, ...parsedVars, createEmptyEnvVar()]
927+
})
928+
scrollToBottom()
936929
}
937930
}
938931

0 commit comments

Comments
 (0)