Skip to content

Remove unused Search results haml files

Terri Chu requested to merge tchu-remove-unused-haml-files into master

What does this MR do and why?

I did some digging and the files used to be referenced in app/views/search/_category.html.haml. I missed them when I removed the file so I think it's safe to remove them now

Screenshots or screen recordings

N/A

Before After

How to set up and validate locally

I validated all searching by doing the following with both new nav and old nav

  1. enabling gdk to use elasticsearch and index everything
  2. enable Advanced search - Admin - Settings - Advanced Search
  3. validate searches still work for the affected scopes (Commits, Comments, Code, Wikis, and Epics)
  4. disabling Advanced search
  5. validate searches still work for the affected scopes (Commits, Comments, Code, Wikis, and Epics)

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 Miguel Rincon

Merge request reports