Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Refactor Final Reports #410

Closed
alejandrosaenz117 opened this issue Oct 27, 2020 · 0 comments · Fixed by #412
Closed

Refactor Final Reports #410

alejandrosaenz117 opened this issue Oct 27, 2020 · 0 comments · Fixed by #412
Assignees
@alejandrosaenz117
Labels
enhancement New feature or request
Milestone
Q4 Goals

Comments

@alejandrosaenz117
Copy link
Collaborator

Now that PrimeNG has been implemented into the front end, the final reports could use a refactoring. Currently, we are displaying Tools & Methodology and the OWASP Risk Rating chart but we understand not everybody uses these items. Therefore, to make the report more universal, we should remove and replace these items with a pie chart and radar chart displaying overall risk severity and vulnerability status.

https://www.primefaces.org/primeng/showcase/#/chart/pie
https://www.primefaces.org/primeng/showcase/#/chart/radar
@alejandrosaenz117 alejandrosaenz117 added the enhancement New feature or request label Oct 27, 2020
@alejandrosaenz117 alejandrosaenz117 added this to the Q4 Goals milestone Oct 27, 2020
@alejandrosaenz117 alejandrosaenz117 self-assigned this Oct 27, 2020
alejandrosaenz117 added a commit that referenced this issue Oct 27, 2020
@alejandrosaenz117
feat(report component): added pie chart and radar chart to report
d955720 
Replaced tools & methodology and OWASP risk rating table with a pie chart and radar chart to finding
status and risk severity.

fix #410
@alejandrosaenz117 alejandrosaenz117 mentioned this issue Oct 27, 2020
Merged
17 tasks
@alejandrosaenz117 alejandrosaenz117 linked a pull request Oct 27, 2020 that will close this issue
Reportchart #412
Merged
17 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alejandrosaenz117 alejandrosaenz117

Labels
enhancement New feature or request
Projects
None yet
Milestone
Q4 Goals
Development

Successfully merging a pull request may close this issue.

Reportchart softrams/bulwark
1 participant
@alejandrosaenz117