Skip to content

Update package list logic for license compliance

- requested to merge 321051-license-compliance-ui-index-out-of-bounds into master

What does this MR do?

Bug Fix for #321051 (closed)

Notes - I couldn't provide screenshots because this was hard to reproduce. The unit tests however and the happy path seem to work fine. The error path is tested in the unit tests.

When there are 0 packages, we just show a blank string for the Component column on the particular row with 0 packages.

Screen_Shot_2021-03-14_at_6.23.31_PM

Screenshots (strongly suggested)

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
Edited by -

Merge request reports