Fix null issueLinks for issue notes
What does this MR do and why?
The MR adds a guard for null nodes
. This will fix the problem:
|
|
---|---|
Screenshots or screen recordings
null
issueLinks
How to set up and validate locally
Setup
- Have the related FF enabled:
echo "Feature.enable(:pipeline_security_dashboard_graphql)" | rails c
- You'll need an EE License
- You'll need to have runners enabled (See $2408961 for setting up a runner)
- Import https://gitlab.com/gitlab-examples/security/security-reports
- Run a pipeline on master
Validation
- Go to the pipeline's security report tab
- Click on a finding with
null
issueLinks
- Verify that the modal renders as expected
Numbered steps to set up and validate the change are strongly suggested.
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Related to #390572 (closed)
Edited by Samantha Ming