Skip to content

Set correct state for Apply button after action

Sam Figueroa requested to merge sf/bugfix/bulk-action-toast-message-goes-wild-421636 into master

What does this MR do and why?

Set correct state for Apply/Remove button after action.

  • Previously the apply action button did not reset after the action was performed and allowed the user to attempt to commit the action again without it making sense to apply. We now default to a reset after commit.

  • This also added a check that the user has rows selected to apply anything to. Without rows selected it will now disable the button.

  • Refs: #421636 (closed)

  • Chore/boy scout rule: add missing emit declarations

Changelog: fixed EE: true

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

Before vs. after video (note: includes narration 🔊 🗣 )

2024-02-15_Screen_Recording_421636Example below:

How to set up and validate locally

  1. Go to compliance center for a group including at least one project
  2. Apply a framework to a project
  3. Apply button should disable
  4. Remove a framework from a project
  5. Remove button should disable and revert to Apply

Merge request reports