Skip to content

Draft: Remove temporary alert dismiss event listener

Alexander Turinske requested to merge remove-temporary-alert-dismiss into master

What does this MR do and why?

Describe in detail what your merge request does and why.

Remove temporary alert dismiss event listener

  • introduced in !41323 (merged), it is no longer needed now that every alert instance in HAML has been converted to the pajamas view component (&5213 (closed))

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

TBD

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

  1. Find an alert that is shown after a page fresh that is dismissible (like in my video)
  2. Verify it dismisses

MR acceptance checklist

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

Edited by Alexander Turinske

Merge request reports

Loading