-
Notifications
You must be signed in to change notification settings - Fork 99
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
Refactor Report Config #4301
Refactor Report Config #4301
Conversation
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Snapshot WarningsEnsure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice. OpenSSF Scorecard
Scanned Files |
We should always use camelCase for all variables.
We agreed on using camelCase naming as it's the standard in the react js world.
Improve the appearance of the checkbox for setting report config parameter to the defaults.
As the report format is saved from the dialog state it needs to be rendered from the dialog state too. We always need a single source of truth for the data.
Always use camelCase variable names.
1e707ee
to
5c7f13d
Compare
Conventional Commits Report
🚀 Conventional commits found. |
🔍 Vulnerabilities of
|
digest | sha256:172ea68bd73d9794d3c3c2cdcbd2c13b4bc6726c508af2405befabf4069691e5 |
vulnerabilities | |
size | 67 MB |
packages | 195 |
📦 Base Image greenbone/gsad:edge
digest | sha256:ab27666918d3214f2d0e43481e3347c0a84a6ac679689e2fbae27b06266495cc |
vulnerabilities |
Description
Description
Description
Description
Description
Description
Description
Description
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Description
Description
Description
Description
Description
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Description
Description
Description
Description
Description
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Description
Description
Description
Description
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Description
Description
Description
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Description
Description
Description
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Description
Description
Description
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Description
Description
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Description
Description
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Description
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Description
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Description
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Description
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Description
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Description
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Description
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Description
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Description
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Description
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Description
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Description
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Description
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Description
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4301 +/- ##
==========================================
- Coverage 62.71% 62.71% -0.01%
==========================================
Files 1120 1120
Lines 99144 99139 -5
Branches 6661 6659 -2
==========================================
- Hits 62180 62175 -5
Misses 36834 36834
Partials 130 130 ☔ View full report in Codecov by Sentry. |
What
Fix displaying the selected report format in the report config dialog and being at it use camelCase for all report config variables.
Why
Get a working report config dialog and always use camelCase for variables.
References
https://jira.greenbone.net/browse/GEA-838
Checklist