Skip to content

Use finding user permissions in finding modal

Lorenz van Herwaarden requested to merge user-permissions-finding-modal into master

What does this MR do and why?

Related #440950 (closed)

This switches from an injected property canAdminVulnerability to using the new userPermissions object on the finding graphql call, and more specifically the property adminVulnerability.

It follows the backend implementation in Add PipelineSecurityReportingFinding userPermis... (#431569 - closed) and the idea that we want the VulnerabilityFindingModal.vue component to be as standalone as possible, and hence not require an injected property.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

How to set up and validate locally

The tests should cover this.

Edited by Lorenz van Herwaarden

Merge request reports