Skip to content

Commit

Permalink
👻 Restore reports page (#1491)
Browse files Browse the repository at this point in the history
- Adds reports page back to side drawer
- Refactors adoption candidate table to use new table format. 
- Started refactoring identified risks, but it doesn't make sense with
our current model to list all questions for a given questionnaire with a
"risk" value and an associated applications column. It just simply
doesn't map to the new model & needs more design input.

https://issues.redhat.com/browse/MTA-1345?filter=-1

---------

Signed-off-by: ibolton336 <[email protected]>
  • Loading branch information
ibolton336 authored Nov 2, 2023
1 parent 1d22ed1 commit aae9115
Show file tree
Hide file tree
Showing 6 changed files with 416 additions and 390 deletions.
2 changes: 1 addition & 1 deletion client/public/locales/en/translation.json
Original file line number Diff line number Diff line change
Expand Up @@ -226,7 +226,7 @@
"archetypesReviewed": "Archetypes reviewed",
"add": "Add",
"additionalNotesOrComments": "Additional notes or comments",
"adoptionCandidateDistribution": "Assessment confidence and risk",
"adoptionCandidateDistribution": "Application confidence and risk",
"affectedApplications": "Affected applications",
"analysis": "Analysis",
"answer": "Answer",
Expand Down
4 changes: 2 additions & 2 deletions client/src/app/layout/SidebarApp/SidebarApp.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -112,14 +112,14 @@ export const SidebarApp: React.FC = () => {
{t("sidebar.archetypes")}
</NavLink>
</NavItem>
{/* <NavItem>
<NavItem>
<NavLink
to={Paths.reports + search}
activeClassName="pf-m-current"
>
{t("sidebar.reports")}
</NavLink>
</NavItem> */}
</NavItem>
<NavItem>
<NavLink to={Paths.controls} activeClassName="pf-m-current">
{t("sidebar.controls")}
Expand Down
Loading

0 comments on commit aae9115

Please sign in to comment.