Fix edge case where users with edit but not read permission were having a strange behavior on the new commit row logic

This commit is contained in:
Carlos Monastyrski
2025-08-05 14:40:21 -03:00
parent 35055955e2
commit 08f7e530b0
@@ -142,7 +142,7 @@ export const SecretItem = memo(
); );
const getDefaultValue = () => { const getDefaultValue = () => {
if (secret.secretValueHidden) { if (secret.secretValueHidden && !isPending) {
return canEditSecretValue ? HIDDEN_SECRET_VALUE : ""; return canEditSecretValue ? HIDDEN_SECRET_VALUE : "";
} }
return secret.valueOverride || secret.value || ""; return secret.valueOverride || secret.value || "";
@@ -413,7 +413,7 @@ export const SecretItem = memo(
tabIndex={0} tabIndex={0}
role="button" role="button"
> >
{secretValueHidden && !isOverriden && ( {secretValueHidden && !isOverriden && !isPending && (
<Tooltip <Tooltip
content={`You do not have access to view the current value${canEditSecretValue && !isRotatedSecret ? ", but you can set a new one" : "."}`} content={`You do not have access to view the current value${canEditSecretValue && !isRotatedSecret ? ", but you can set a new one" : "."}`}
> >
@@ -444,12 +444,14 @@ export const SecretItem = memo(
<InfisicalSecretInput <InfisicalSecretInput
isReadOnly={isReadOnlySecret} isReadOnly={isReadOnlySecret}
key="secret-value" key="secret-value"
isVisible={isVisible && !secretValueHidden} isVisible={isVisible && (!secretValueHidden || isPending)}
canEditButNotView={secretValueHidden && !isOverriden} canEditButNotView={secretValueHidden && !isOverriden && !isPending}
environment={environment} environment={environment}
secretPath={secretPath} secretPath={secretPath}
{...field} {...field}
defaultValue={secretValueHidden ? HIDDEN_SECRET_VALUE : undefined} defaultValue={
secretValueHidden && !isPending ? HIDDEN_SECRET_VALUE : undefined
}
containerClassName="py-1.5 rounded-md transition-all" containerClassName="py-1.5 rounded-md transition-all"
/> />
)} )}