Merge pull request #324 from bcgov/oleks

hotfix - platform users should not see listings link in top-menu (DSS-440)
bcgov · May 29, 2024 · 25cb6d1 · 25cb6d1
2 parents 29db5d6 + fa1907d
commit 25cb6d1
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 10 deletions.
diff --git a/frontend/src/app/app.routes.ts b/frontend/src/app/app.routes.ts
@@ -52,12 +52,12 @@ export const routes: Routes = [
         component: DelistingRequestComponent,
         data: { permissions: [takedown_action] }
     },
-    {
-        path: 'listings',
-        canActivate: [approvedUserGuard, activeUserGuard, hasPermissionsGuard, areTermsAceptedGuard],
-        component: ListingsTableComponent,
-        data: { permissions: [listing_read] }
-    },
+    // {
+    //     path: 'listings',
+    //     canActivate: [approvedUserGuard, activeUserGuard, hasPermissionsGuard, areTermsAceptedGuard],
+    //     component: ListingsTableComponent,
+    //     data: { permissions: [listing_read] }
+    // },
     {
         path: 'access-request',
         canActivate: [accessRequestTokenGuard],

diff --git a/frontend/src/app/common/layout/layout.component.ts b/frontend/src/app/common/layout/layout.component.ts
@@ -13,6 +13,7 @@ import { ErrorHandlingService } from '../services/error-handling.service';
 import { ErrorBackEnd } from '../models/errors';
 import { TopMenuService } from '../services/top-menu.service';
 import { TooltipModule } from 'primeng/tooltip';
+import { listing_read } from '../consts/permissions.const';
 
 @Component({
   selector: 'app-layout',
@@ -78,10 +79,12 @@ export class LayoutComponent {
       return !!item.items?.length;
     });
 
-    this.items?.unshift({
-      label: 'Listings',
-      routerLink: '/listings',
-    });
+    // if (this.userDataService.currentUser.permissions.includes(listing_read)) {
+    //   this.items?.unshift({
+    //     label: 'Listings',
+    //     routerLink: '/listings',
+    //   });
+    // }
 
     this.items?.unshift({
       label: 'Home',