Skip to content

Comments

Feature: Report Page#495

Open
sawankshrma wants to merge 1 commit intodevsecopsmaturitymodel:mainfrom
sawankshrma:new-feat-Report
Open

Feature: Report Page#495
sawankshrma wants to merge 1 commit intodevsecopsmaturitymodel:mainfrom
sawankshrma:new-feat-Report

Conversation

@sawankshrma
Copy link

Description

A new "Report" page is created.
It "dumps" all activities at one page in order to provide it to an auditor, Preferably in a pdf format.

For now, a slightly modified version of the already existing ActivityDescriptionComponent is used in this page to display all/some activities on the page. => This is of-course not exportable/printable for now and will be fixed in later iterations.

The activities (types, individual) and their attributes are configurable as to what we'd like to show in the report and eventually provide to the auditor. The configurations are stored in the localStorage in the browser like all other informations.

Resolves Issue: #494

image

What's Changed?

  • A new ReportComponent is created which dumps all the activities rendered in a modified ActivityDescriptionComponent called ReportConfigModalComponent --> this component is going to be replaced by something better which can be exportable.
  • It contains a button to Adjust the Report's Configuration which opens a modal whose component is also a newly generated component called ReportConfigModalComponent. All the configurations of the report can be altered through this modal. The new configurations are then modified and saved in the localStorage and the Report page is then re-rendered with those modified configurations.
  • A new model file called report-config.ts is created which exports some functions and interfaces useful to the "Report" page.
  • The app-routing.module.ts and sidenav-buttons.component.ts files are modified to contain the new "Report" page.

showCloseButton = false;

currentActivity: Partial<Activity> = {};
TimeLabel: string = '';
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it will be better to implement Activity (https://github.com/devsecopsmaturitymodel/DevSecOps-MaturityModel/blob/main/src/app/model/activity-store.ts#L45) and use the variable names from the Activity.

Copy link
Author

@sawankshrma sawankshrma Feb 21, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Got It. Will do the necessary changes in a follow up commit. Will resolve this afterwards...

@wurstbrot
Copy link
Collaborator

Than you @sawankshrma , comments in the review.

@wurstbrot wurstbrot requested review from 0x41head and vbakke February 21, 2026 09:56
@sawankshrma
Copy link
Author

Than you @sawankshrma , comments in the review.

HI @wurstbrot . Thank You for the review.

See, this component (ReportActivityComponent) was not built by me from Scratch. I just copy-pasted the already existing ActivityDescriptionComponent's code. I just changed some lines and/or commented some lines which were causing trouble in the "Report" Page. That's why it has some commented code and some extra lines which are not required here.

In further iterations I am going to replace this component completely to provide for an Exportable/Printable "Report" page. I hope you understand :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants