Skip to content

Add polling to security training service graphql query

Samantha Ming requested to merge 355132-graphql-polling into master

What does this MR do and why?

Because the security urls are async, we will need to add polling to our GraphQL query on the frontend 👍

This MR does not introduce any UI/UX change

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

  1. Navigate to the Vulnerability report
  2. Click on a vulnerability that is of type of CWE
  3. Open the network tab
  4. You will see an initial graphql request while the loading state appears
  5. You will see subsequent graphql request made every 5 secs until the training urls appears
Pending Completed
image image

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 #355132 (closed)

Edited by Samantha Ming

Merge request reports