From e6a569b153f72b9f97838834c04ec9d7db239b2a Mon Sep 17 00:00:00 2001 From: SmithThe4th Date: Tue, 5 Mar 2024 09:38:38 -0500 Subject: [PATCH] [PM-6584] [PM-6632] [Defects] Vertical Vault Refresh Product Switcher (#8198) * removed org enabled check from acOrg logic * fixed bug where organization doesn't show on admin console for users admin custom permissions --- .../organizations/layouts/organization-layout.component.ts | 3 ++- .../product-switcher/product-switcher-content.component.ts | 4 ++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/apps/web/src/app/admin-console/organizations/layouts/organization-layout.component.ts b/apps/web/src/app/admin-console/organizations/layouts/organization-layout.component.ts index 81b1ca142b..90010160aa 100644 --- a/apps/web/src/app/admin-console/organizations/layouts/organization-layout.component.ts +++ b/apps/web/src/app/admin-console/organizations/layouts/organization-layout.component.ts @@ -8,6 +8,7 @@ import { canAccessBillingTab, canAccessGroupsTab, canAccessMembersTab, + canAccessOrgAdmin, canAccessReportingTab, canAccessSettingsTab, canAccessVaultTab, @@ -43,7 +44,7 @@ import { AdminConsoleLogo } from "../../icons/admin-console-logo"; export class OrganizationLayoutComponent implements OnInit, OnDestroy { protected readonly logo = AdminConsoleLogo; - protected orgFilter = (org: Organization) => org.isAdmin; + protected orgFilter = (org: Organization) => canAccessOrgAdmin(org); organization$: Observable; showPaymentAndHistory$: Observable; diff --git a/apps/web/src/app/layouts/product-switcher/product-switcher-content.component.ts b/apps/web/src/app/layouts/product-switcher/product-switcher-content.component.ts index 5354dc2b71..0ce98948c9 100644 --- a/apps/web/src/app/layouts/product-switcher/product-switcher-content.component.ts +++ b/apps/web/src/app/layouts/product-switcher/product-switcher-content.component.ts @@ -58,9 +58,9 @@ export class ProductSwitcherContentComponent { // If the active route org doesn't have access to AC, find the first org that does. const acOrg = - routeOrg != null && canAccessOrgAdmin(routeOrg) && routeOrg.enabled + routeOrg != null && canAccessOrgAdmin(routeOrg) ? routeOrg - : orgs.find((o) => canAccessOrgAdmin(o) && o.enabled); + : orgs.find((o) => canAccessOrgAdmin(o)); // TODO: This should be migrated to an Observable provided by the provider service and moved to the combineLatest above. See AC-2092. const providers = await this.providerService.getAll();