Merge pull request #2630 from scott-ray-wilson/incorrect-import-value-display-fix

Fix: Correct Secret Value Override Display on Overview Table
This commit is contained in:
Maidul Islam
2024-10-21 14:18:43 -04:00
committed by GitHub
4 changed files with 23 additions and 7 deletions
+7 -5
View File
@@ -48,7 +48,7 @@ export const fetchProjectSecrets = async ({
}; };
export const mergePersonalSecrets = (rawSecrets: SecretV3Raw[]) => { export const mergePersonalSecrets = (rawSecrets: SecretV3Raw[]) => {
const personalSecrets: Record<string, { id: string; value?: string }> = {}; const personalSecrets: Record<string, { id: string; value?: string; env: string }> = {};
const secrets: SecretV3RawSanitized[] = []; const secrets: SecretV3RawSanitized[] = [];
rawSecrets.forEach((el) => { rawSecrets.forEach((el) => {
const decryptedSecret: SecretV3RawSanitized = { const decryptedSecret: SecretV3RawSanitized = {
@@ -69,7 +69,8 @@ export const mergePersonalSecrets = (rawSecrets: SecretV3Raw[]) => {
if (el.type === SecretType.Personal) { if (el.type === SecretType.Personal) {
personalSecrets[decryptedSecret.key] = { personalSecrets[decryptedSecret.key] = {
id: el.id, id: el.id,
value: el.secretValue value: el.secretValue,
env: el.environment
}; };
} else { } else {
secrets.push(decryptedSecret); secrets.push(decryptedSecret);
@@ -77,9 +78,10 @@ export const mergePersonalSecrets = (rawSecrets: SecretV3Raw[]) => {
}); });
secrets.forEach((sec) => { secrets.forEach((sec) => {
if (personalSecrets?.[sec.key]) { const personalSecret = personalSecrets?.[sec.key];
sec.idOverride = personalSecrets[sec.key].id; if (personalSecret && personalSecret.env === sec.env) {
sec.valueOverride = personalSecrets[sec.key].value; sec.idOverride = personalSecret.id;
sec.valueOverride = personalSecret.value;
sec.overrideAction = "modified"; sec.overrideAction = "modified";
} }
}); });
@@ -335,7 +335,12 @@ export const OrgGroupsTable = ({ handlePopUpOpen }: Props) => {
})} })}
</TBody> </TBody>
</Table> </Table>
{groups?.length === 0 && <EmptyState title="No groups found" icon={faUsers} />} {filteredGroups?.length === 0 && (
<EmptyState
title={groups?.length === 0 ? "No groups found" : "No groups match search"}
icon={faUsers}
/>
)}
</TableContainer> </TableContainer>
</div> </div>
); );
@@ -377,7 +377,14 @@ export const OrgMembersTable = ({ handlePopUpOpen, setCompleteInviteLinks }: Pro
</TBody> </TBody>
</Table> </Table>
{!isLoading && filterdUser?.length === 0 && ( {!isLoading && filterdUser?.length === 0 && (
<EmptyState title="No organization members found" icon={faUsers} /> <EmptyState
title={
members?.length === 0
? "No organization members found"
: "No organization members match search"
}
icon={faUsers}
/>
)} )}
</TableContainer> </TableContainer>
</div> </div>
@@ -524,6 +524,8 @@ export const SecretOverviewPage = () => {
); );
const allRowsSelectedOnPage = useMemo(() => { const allRowsSelectedOnPage = useMemo(() => {
if (!secrets?.length && !folders?.length) return { isChecked: false, isIndeterminate: false };
if ( if (
(!secrets?.length || (!secrets?.length ||
secrets?.every((secret) => selectedEntries[EntryType.SECRET][secret.key])) && secrets?.every((secret) => selectedEntries[EntryType.SECRET][secret.key])) &&