Allow users to type the same original value on bulk commits and remove them if no changes are left

This commit is contained in:
Carlos Monastyrski
2025-08-04 18:22:30 -03:00
parent e8eb1b5f8b
commit 3e803debb4
2 changed files with 13 additions and 26 deletions
@@ -222,7 +222,7 @@ export const SecretItem = memo(
} }
if (isDirty && !isSubmitting && !isAutoSavingRef.current) { if (isDirty && !isSubmitting && !isAutoSavingRef.current) {
const debounceTime = 600; const debounceTime = 200;
autoSaveTimeoutRef.current = setTimeout(() => { autoSaveTimeoutRef.current = setTimeout(() => {
autoSaveChanges(formValues); autoSaveChanges(formValues);
@@ -343,31 +343,18 @@ export const SecretListView = ({
id: orgSecret.id, id: orgSecret.id,
type: PendingAction.Update, type: PendingAction.Update,
secretKey: trueOriginalSecret.key, secretKey: trueOriginalSecret.key,
...(key !== trueOriginalSecret.key && { newSecretName: key }), newSecretName: key,
...(value !== trueOriginalSecret.value && {
originalValue: trueOriginalSecret.value, originalValue: trueOriginalSecret.value,
secretValue: value secretValue: value,
}),
...(comment !== trueOriginalSecret.comment && {
originalComment: trueOriginalSecret.comment, originalComment: trueOriginalSecret.comment,
secretComment: comment secretComment: comment,
}),
...(modSecret.skipMultilineEncoding !==
trueOriginalSecret.skipMultilineEncoding && {
originalSkipMultilineEncoding: trueOriginalSecret.skipMultilineEncoding, originalSkipMultilineEncoding: trueOriginalSecret.skipMultilineEncoding,
skipMultilineEncoding: modSecret.skipMultilineEncoding skipMultilineEncoding: modSecret.skipMultilineEncoding,
}),
...(!isSameTags && {
originalTags: originalTags:
trueOriginalSecret.tags?.map((tag) => ({ id: tag.id, slug: tag.slug })) || [], trueOriginalSecret.tags?.map((tag) => ({ id: tag.id, slug: tag.slug })) || [],
tags: tags?.map((tag) => ({ id: tag.id, slug: tag.name || tag.slug || "" })) || [] tags: tags?.map((tag) => ({ id: tag.id, slug: tag.name || tag.slug || "" })) || [],
}),
...(JSON.stringify(secretMetadata) !==
JSON.stringify(trueOriginalSecret.secretMetadata) && {
originalSecretMetadata: trueOriginalSecret.secretMetadata || [], originalSecretMetadata: trueOriginalSecret.secretMetadata || [],
secretMetadata: secretMetadata || [] secretMetadata: secretMetadata || [],
}),
timestamp: Date.now(), timestamp: Date.now(),
resourceType: "secret", resourceType: "secret",
existingSecret: orgSecret existingSecret: orgSecret