From 764f4703e00b9f50afbc5ca35f0da50dd1c9dc12 Mon Sep 17 00:00:00 2001 From: Devon Anderson Date: Tue, 6 Dec 2022 13:37:55 -0500 Subject: [PATCH] =?UTF-8?q?chore:=20=F0=9F=A4=96=20brokered=20and=20inject?= =?UTF-8?q?ed=20credential=20associations=20(#1512)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * chore: 🤖 brokered and injected credential associations Verified JSON credentials show up as a source that can be added to a target as `brokered`, and not show up as a source that can be added to a target as `injected`. ✅ Closes: ICU-6833, ICU-6834, ICU-6835, ICU-6836 * Update ui/admin/app/controllers/scopes/scope/targets/target/add-injected-application-credential-sources.js Co-authored-by: Randall Morey Co-authored-by: Randall Morey --- .../target/add-injected-application-credential-sources.js | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/ui/admin/app/controllers/scopes/scope/targets/target/add-injected-application-credential-sources.js b/ui/admin/app/controllers/scopes/scope/targets/target/add-injected-application-credential-sources.js index 3033b287b..fbfaceff6 100644 --- a/ui/admin/app/controllers/scopes/scope/targets/target/add-injected-application-credential-sources.js +++ b/ui/admin/app/controllers/scopes/scope/targets/target/add-injected-application-credential-sources.js @@ -27,12 +27,11 @@ export default class ScopesScopeTargetsTargetAddInjectedApplicationCredentialSou (source) => source.value ) ); - const notAddedCredentialLibraries = this.model.credentialLibraries.filter( ({ id }) => !currentCredentialSourceIDs.has(id) ); const notAddedCredentials = this.model.credentials.filter( - ({ id }) => !currentCredentialSourceIDs.has(id) + ({ id, type }) => !currentCredentialSourceIDs.has(id) && type !== 'json' ); return [...notAddedCredentialLibraries, ...notAddedCredentials]; }