Skip to content

Rename first_class_init to vulnerability_report_init

Savas Vedova requested to merge 321775-rename-first-class-init into master

What does this MR do?

This MR simply renames first_class_init to vulnerability_report_init.

Why are we doing this change?

When we first rolled out the refactored security dashboards we named the vulnerabilities as first_class_vulnerabilities. This is no longer the case. As part of #321775 (closed), I thought that now it makes sense to rename these files to vulnerability_report which reflects the real page name.

Screenshots (strongly suggested)

Nothing should change visually, but here's a screenshot for all three dashboards which shows that they're still working:

dashboard screenshot
group Screenshot_2021-03-24_at_12.54.49
project Screenshot_2021-03-24_at_12.54.34
instance Screenshot_2021-03-24_at_12.55.01

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team

Related to #321775 (closed)

Merge request reports

Loading