Skip to content

Migrate MR Widget Suggest pipeline component from DismissibleContainer to UserCalloutDismisser

Buck O'Learyrequested to merge
576356-remove-dismissable-container-component into master

What does this MR do and why?

Migrates MR Widget Suggest Pipeline to utilize UserCalloutDismisser

  • Migrates mr_widget_suggest_pipeline from DismissibleContainer to UserCalloutDismisser
  • Removes deprecated dismissible_container.vue component and associated tests
  • Transitions dismissal logic from REST API (axios) to GraphQL
  • Simplifies component by removing userCalloutsPath prop requirement
  • Updates test suite to use makeMockUserCalloutDismisser helper for consistency
  • Maintains backwards compatibility with existing dismissal behavior

References

Screenshots or screen recordings

No UI changes.

Before After

How to set up and validate locally

  1. Create a new project (without any CI/CD)
  2. Create a brand new merge request in your new project
  3. Make sure you have Maintainer or Owner permissions
  4. Ensure there's no .gitlab-ci.yml file anywhere in the repository
  5. Navigate to the MR page - the component should appear - dismiss the component and ensure the dismissal works correctly (alternatively you can verify in the rails console by finding your user and checking the output of user.callouts)

MR acceptance checklist

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

Related to #576356

Edited by Buck O'Leary

Merge request reports