Skip to content

Commit

Permalink
Merge pull request #2238 from cisagov/2237-hide-reports-page-1
Browse files Browse the repository at this point in the history
Updated header (Nav Bar) for hiding reports page
  • Loading branch information
courtneyc1 authored Sep 20, 2023
2 parents 9445de8 + 573bcfc commit ef2c654
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 13 deletions.
6 changes: 5 additions & 1 deletion frontend/src/components/Header.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -247,12 +247,16 @@ const HeaderNoCtx: React.FC<ContextType> = (props) => {
exact: false
},
{ title: 'Feeds', path: '/feeds', users: ALL_USERS, exact: false },

/*
Hiding Reports page until finished
{
title: 'Reports',
path: '/reports',
users: ALL_USERS,
exact: true
},
},*/

{
title: 'Scans',
path: '/scans',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -60,17 +60,6 @@ exports[`Header component matches snapshot 1`] = `
Feeds
</a>
</div>
<div
class="css-g2ra7n"
>
<a
class="NavItem-link"
href="/reports"
style="outline: none;"
>
Reports
</a>
</div>
</div>
<div
class="Header-spacing"
Expand Down
3 changes: 2 additions & 1 deletion frontend/src/test-utils/organization.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,5 +9,6 @@ export const testOrganization = {
granularScans: [],
tags: [],
parent: null,
children: []
children: [],
pendingDomains: []
};

0 comments on commit ef2c654

Please sign in to comment.