Merge pull request #1948 from Infisical/fix/resolved-import-override-display

fix: resolved import override behavior
This commit is contained in:
Sheen Capadngan
2024-06-11 15:55:19 +08:00
committed by GitHub
@@ -45,7 +45,6 @@ export const computeImportedSecretRows = (
if (importedSecIndex === -1) return []; if (importedSecIndex === -1) return [];
const importedSec = importSecrets[importedSecIndex]; const importedSec = importSecrets[importedSecIndex];
const overridenSec: Record<string, { env: string; secretPath: string }> = {}; const overridenSec: Record<string, { env: string; secretPath: string }> = {};
for (let i = importedSecIndex + 1; i < importSecrets.length; i += 1) { for (let i = importedSecIndex + 1; i < importSecrets.length; i += 1) {
@@ -61,11 +60,28 @@ export const computeImportedSecretRows = (
overridenSec[el.key] = { env: SECRET_IN_DASHBOARD, secretPath: "" }; overridenSec[el.key] = { env: SECRET_IN_DASHBOARD, secretPath: "" };
}); });
return importedSec.secrets.map(({ key, value }) => ({ const importedEntry: Record<string, boolean> = {};
key, const importedSecretEntries: {
value, key: string;
overriden: overridenSec?.[key] value: string;
})); overriden: {
env: string;
secretPath: string;
};
}[] = [];
importedSec.secrets.forEach(({ key, value }) => {
if (!importedEntry[key]) {
importedSecretEntries.push({
key,
value,
overriden: overridenSec?.[key]
});
importedEntry[key] = true;
}
});
return importedSecretEntries;
}; };
type Props = { type Props = {
@@ -159,8 +175,9 @@ export const SecretImportListView = ({
importEnv.slug === environment && importEnv.slug === environment &&
isReserved && isReserved &&
importPath === importPath ===
`${secretPath === "/" ? "" : secretPath}/${ReservedFolders.SecretReplication `${secretPath === "/" ? "" : secretPath}/${
}${replicationImportId}` ReservedFolders.SecretReplication
}${replicationImportId}`
); );
if (reservedImport) { if (reservedImport) {
setReplicationSecrets((state) => ({ setReplicationSecrets((state) => ({
@@ -206,8 +223,9 @@ export const SecretImportListView = ({
isOpen={popUp.deleteSecretImport.isOpen} isOpen={popUp.deleteSecretImport.isOpen}
deleteKey="unlink" deleteKey="unlink"
title="Do you want to remove this secret import?" title="Do you want to remove this secret import?"
subTitle={`This will unlink secrets from environment ${(popUp.deleteSecretImport?.data as TSecretImport)?.importEnv subTitle={`This will unlink secrets from environment ${
} of path ${(popUp.deleteSecretImport?.data as TSecretImport)?.importPath}?`} (popUp.deleteSecretImport?.data as TSecretImport)?.importEnv
} of path ${(popUp.deleteSecretImport?.data as TSecretImport)?.importPath}?`}
onChange={(isOpen) => handlePopUpToggle("deleteSecretImport", isOpen)} onChange={(isOpen) => handlePopUpToggle("deleteSecretImport", isOpen)}
onDeleteApproved={handleSecretImportDelete} onDeleteApproved={handleSecretImportDelete}
/> />