Skip to content

Fix search helper vertical navigation json method

Tomas Bulva requested to merge fix-search-helper-json-blob-condition into master

What does this MR do and why?

The condition on the vertical navigation JSON generator had minor logical issues.

4/7 add vertical navigation vue app

Screenshots or screen recordings

This is a technical depth change there are no visual changes.

Screenshot_2022-11-08_at_14.02.43

How to set up and validate locally

  1. checkout this branch
  2. In the rails console enable the feature flag
    Feature.enable(:search_page_vertical_nav)
  3. disable "search with Elasticsearch enabled" (http://127.0.0.1:3000/admin/application_settings/advanced_search)
    Screenshot_2022-11-08_at_13.54.19
  4. search any term on the search page http://127.0.0.1:3000/search
  5. code search should not appear

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

Edited by Tomas Bulva

Merge request reports