fix: approvals triggering for personal secrets

This commit is contained in:
Daniel Hougaard
2025-01-25 03:44:43 +01:00
parent c12c6dcc6e
commit 860eaae4c8
@@ -180,10 +180,12 @@ export const SecretListView = ({
try { try {
// personal secret change // personal secret change
let personalAction = false;
if (overrideAction === "deleted") { if (overrideAction === "deleted") {
await handleSecretOperation("delete", SecretType.Personal, oldKey, { await handleSecretOperation("delete", SecretType.Personal, oldKey, {
secretId: orgSecret.idOverride secretId: orgSecret.idOverride
}); });
personalAction = true;
} else if (overrideAction && idOverride) { } else if (overrideAction && idOverride) {
await handleSecretOperation("update", SecretType.Personal, oldKey, { await handleSecretOperation("update", SecretType.Personal, oldKey, {
value: valueOverride, value: valueOverride,
@@ -191,14 +193,16 @@ export const SecretListView = ({
secretId: orgSecret.idOverride, secretId: orgSecret.idOverride,
skipMultilineEncoding: modSecret.skipMultilineEncoding skipMultilineEncoding: modSecret.skipMultilineEncoding
}); });
personalAction = true;
} else if (overrideAction) { } else if (overrideAction) {
await handleSecretOperation("create", SecretType.Personal, oldKey, { await handleSecretOperation("create", SecretType.Personal, oldKey, {
value: valueOverride value: valueOverride
}); });
personalAction = true;
} }
// shared secret change // shared secret change
if (!isSharedSecUnchanged) { if (!isSharedSecUnchanged && !personalAction) {
await handleSecretOperation("update", SecretType.Shared, oldKey, { await handleSecretOperation("update", SecretType.Shared, oldKey, {
value, value,
tags: tagIds, tags: tagIds,
@@ -232,10 +236,11 @@ export const SecretListView = ({
}); });
handlePopUpClose("secretDetail"); handlePopUpClose("secretDetail");
createNotification({ createNotification({
type: isProtectedBranch ? "info" : "success", type: isProtectedBranch && !personalAction ? "info" : "success",
text: isProtectedBranch text:
? "Requested changes have been sent for review" isProtectedBranch && !personalAction
: "Successfully saved secrets" ? "Requested changes have been sent for review"
: "Successfully saved secrets"
}); });
} catch (error) { } catch (error) {
console.log(error); console.log(error);