Refactor aggregations calls for issues
What does this MR do and why?
Related to #461874 (closed)
Behind feature flag issues_search_refactor
and broken up in a few MRs to reduce review complexity and risk:
Description | MR Link |
---|---|
Refactor Empty search results class | !157210 (merged) |
Rubocop fixes for search_results_spec.rb
|
!157517 (merged) |
Refactor search calls for issues | !155516 (merged) |
Refactor aggregations calls for issues |
|
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
Labels shown in drop down
Label selected
Before | After |
---|---|
How to set up and validate locally
test with and without the show_issue_refactor
feature flag
- enable
elasticsearch
for gdk and index everything usinggitlab:elastic:index
rake task - apply labels to a few issues
- perform issue search
- verify that labels load in the drop down, and that they match the labels from the results
- select one of the labels and apply
- ensure the labels filter is applied to the search results
Edited by Terri Chu