Add support for split button loading
What does this MR do and why?
This MR updates how the split_button.vue
renders its loading state -- switched to use a more simplified approach with a loading
prop instead, which aligns with how we set the disabled state with the disabled
prop. This way it will be easier to handle the loading state without the need to iterate through the buttons array
MR acceptance checklist
Before | After |
---|---|
How to set up and validate locally
- Have the related FF enabled:
echo "Feature.enable(:resolve_vulnerability_ai)" | rails c
- In order to run a pipeline to generate a Vulnerability report, you'll need an EE license.
- Fork https://gitlab.com/gitlab-org/govern/demos/sandbox/minac/test-remediations
- Run a pipeline against the default branch. It will generate vulnerabilities.
- Go to the project vulnerability report page
- Click on the SAST vulnerability
- In the dropdown down select the "Resolve with AI" button
- When clicked, the button should display a loading state
Note: the demo repo doesn't contain real vulnerabilities, so if you try to click on it, it will successfully make the request but you will get an error when trying to generate AI MR. This is okay. We're only testing the UI here.
Related to #434940 (closed)
Edited by Samantha Ming