Skip to content

Move explain vulnerability feedback to bottom of AI explanation

Daniel Tian requested to merge 417612-move-explain-vulnerability-feedback into master

What does this MR do and why?

This MR moves the user feedback in the Explain Vulnerability drawer from pinned at the bottom of the drawer, to the bottom of the AI response. It also updates the textarea placeholder text from Explain your rating (optional) to Explain your rating to help us improve! (optional).

Before After
Peek_2023-09-12_04-38 Peek_2023-09-12_04-37

How to set up and validate locally

  1. Clone a project that has vulnerabilities like WebGoat.NET.
  2. Run a pipeline against the master branch.
  3. Go to Security and Compliance -> Vulnerability Report and click on any SAST vulnerability.
  4. Click on Try it out in the Explain Vulnerability card.
  5. Verify that the user feedback is at the bottom of the AI response and scrolls with the content, instead of pinned at the bottom of the drawer.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #417612 (closed)

Edited by Daniel Tian

Merge request reports