Merge branch '8370-view-issue-button-doesn-t-work-in-dashboard-modal' into 'master'

Resolve "View issue button doesn't work in dashboard modal"

Closes #8370

See merge request gitlab-org/gitlab-ee!8385

ee/app/assets/javascripts/vue_shared/security_reports/components/modal.vue

@@ -249,7 +249,7 @@ export default {
 
         <a
           v-if="modal.vulnerability.hasIssue"
-          :href="modal.vulnerability.issueFeedback && modal.vulnerability.issueFeedback.issue_url"
+          :href="modal.vulnerability.issue_feedback && modal.vulnerability.issue_feedback.issue_url"
           rel="noopener noreferrer nofollow"
           class="btn btn-success btn-inverted"
         >

ee/app/assets/javascripts/vue_shared/security_reports/store/utils.js

@@ -31,7 +31,7 @@ function enrichVulnerabilityWithfeedback(vulnerability, feedback = []) {
         return {
           ...vuln,
           hasIssue: true,
-          issueFeedback: fb,
+          issue_feedback: fb,
         };
       }
       return vuln;

ee/changelogs/unreleased/8370-view-issue-button-doesn-t-work-in-dashboard-modal.yml

+---
+title: Fixes the view issue button in the Group Security Dashboard
+merge_request: 8385
+author:
+type: fixed

ee/spec/javascripts/vue_shared/security_reports/store/utils_spec.js

@@ -87,7 +87,7 @@ describe('security reports utils', () => {
       expect(parsed.hasIssue).toEqual(true);
       expect(parsed.isDismissed).toEqual(true);
       expect(parsed.dismissalFeedback).toEqual(sastFeedbacks[0]);
-      expect(parsed.issueFeedback).toEqual(sastFeedbacks[1]);
+      expect(parsed.issue_feedback).toEqual(sastFeedbacks[1]);
     });
   });
 
@@ -129,7 +129,7 @@ describe('security reports utils', () => {
       expect(parsed.hasIssue).toEqual(true);
       expect(parsed.isDismissed).toEqual(true);
       expect(parsed.dismissalFeedback).toEqual(dependencyScanningFeedbacks[0]);
-      expect(parsed.issueFeedback).toEqual(dependencyScanningFeedbacks[1]);
+      expect(parsed.issue_feedback).toEqual(dependencyScanningFeedbacks[1]);
     });
   });
 
@@ -165,7 +165,7 @@ describe('security reports utils', () => {
       expect(parsed.hasIssue).toEqual(true);
       expect(parsed.isDismissed).toEqual(true);
       expect(parsed.dismissalFeedback).toEqual(containerScanningFeedbacks[0]);
-      expect(parsed.issueFeedback).toEqual(containerScanningFeedbacks[1]);
+      expect(parsed.issue_feedback).toEqual(containerScanningFeedbacks[1]);
     });
   });
 
@@ -180,7 +180,7 @@ describe('security reports utils', () => {
       expect(parsed.hasIssue).toEqual(true);
       expect(parsed.isDismissed).toEqual(true);
       expect(parsed.dismissalFeedback).toEqual(dastFeedbacks[0]);
-      expect(parsed.issueFeedback).toEqual(dastFeedbacks[1]);
+      expect(parsed.issue_feedback).toEqual(dastFeedbacks[1]);
     });
   });