-
Notifications
You must be signed in to change notification settings - Fork 59
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.
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
Findings dashboard for all organizations #4007
base: main
Are you sure you want to change the base?
Conversation
…minvws/nl-kat-coordination into feature/findings-dashboard-for-all-orgs
…ture/findings-dashboard-for-all-orgs
…hub.com/minvws/nl-kat-coordination into feature/findings-dashboard-for-all-orgs
…minvws/nl-kat-coordination into feature/findings-dashboard-for-all-orgs
…minvws/nl-kat-coordination into feature/findings-dashboard-for-all-orgs
…minvws/nl-kat-coordination into feature/findings-dashboard-for-all-orgs
…minvws/nl-kat-coordination into feature/findings-dashboard-for-all-orgs
…minvws/nl-kat-coordination into feature/findings-dashboard-for-all-orgs
QA suggestion before I forget it completely. The QA notes mention the following:
I think it is a good practice to add a little 'howto' with all the copy paste steps to do this, to the docs. As to make it easier for user to migrate and activate this. |
Found the following 4 things for a clean install on this PR.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few suggestions:
- Based on the
dashboards
make target and the output of this command,dashboards
would be a better command name. I would suggest renaming this module todashboards.py
. - There's a lot happening here when the command is run, but there are no specific logs or prints that give some feedback to the user. I would expect at least something like "Creating dashboard for organization X..." when the command is creating all the dashboards
@register.filter | ||
def get_date(date_str: str) -> datetime: | ||
return datetime.fromisoformat(date_str) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Return type indicates that this is a date time type, not a date.
@register.filter | |
def get_date(date_str: str) -> datetime: | |
return datetime.fromisoformat(date_str) | |
@register.filter | |
def get_date(date_str: str) -> date: | |
return datetime.fromisoformat(date_str).date |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@madelondohmen is the idea here to return a datetime object? if yes, we should make this function name: get_datetime instead. The function name says to return a date only object as that was the intention or renaming the function to return indeed what it was for .
Co-authored-by: ammar92 <[email protected]>
…minvws/nl-kat-coordination into feature/findings-dashboard-for-all-orgs
The dos has been updated |
Running |
Quality Gate failedFailed conditions |
Changes
This PR adds dashboards to OpenKAT.
In the frontend we've only added the Findings Dashboard to the Crisis Room of all organizations, but the backend is already ready to add all kinds of dashboards to both Crisis Rooms.
This PR adds:
Issue link
Closes #3881
Demo
Opname.2025-01-07.102915.mp4
Aggregate Report:
QA notes
New organization
Create a new organization. Creating a new organization would automatically add this organization to the Findings Dashboard (which is shown in the Crisis Room for all organizations). Besides this, on the Scheduled Reports page there should be a ReportRecipe called "Crisis Room Aggregate Report". This should also create a report, which can be seen on the Report History page (this may take a minute). After the report has been created, the results of the report should be visible on the Crisis Room for all organizations. Please make sure that these two matches.
Existing organizations
Existing organizations also need their own dashboard. This can be done by using the command "make dashboard" in rocky. Please check if this adds the existing organizations to the Findings Dashboard (which is shown in the Crisis Room for all organizations).
Findings Report
The Findings Report should work as expected. Also, it should be possible to create a Aggregate and Multi Findings Report. Please check if this work as expected.
Code Checklist
.env
changes files if required and changed the.env-dist
accordingly.Checklist for code reviewers:
Copy-paste the checklist from the docs/source/templates folder into your comment.
Checklist for QA:
Copy-paste the checklist from the docs/source/templates folder into your comment.