Skip to content

Replace issuable sort dropdown

Serhii Yarynovskyi requested to merge sy-replace-issuable-sort-dropdown into master

What does this MR do and why?

One or more dropdowns in app/views/shared/issuable/_sort_dropdown.html.haml are not Pajamas-compliant. See the parent epic for guidance on migration strategies.

Initially was done in: !81430 (merged) but because of an error: !82322 (merged) was reverted: !82558 (merged) Since error is fixed, should be fine now

Screenshots or screen recordings

Before After
Screenshot_2022-05-05_at_16.42.14 Screenshot_2022-05-05_at_16.41.43

How to set up and validate locally

  1. Go to the project with multiple merge requests.
  2. Open merge request.
  3. Sort by Merged date
  4. Go to dashboard merge requests and make sure page is loading

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

Edited by Serhii Yarynovskyi

Merge request reports