Skip to content

Remove switch to basic search link

Dylan Griffith requested to merge remove-switch-to-basic-search-link into master

What does this MR do?

This was originally implemented as a back up because we were concerned about the reliability of Elasticsearch. We assumed it would be safe to add this link just in case our Elasticsearch experience was consistently bad.

But it turned out that we forgot to ever turn the feature flag on and our Elasticsearch has been quite robust anyway. As such we may as well remove the feature now.

We still keep the hidden feature where the user can manually add a basic_search=true to the URL to get the basic search functionality because this is still useful for diagnostic purposes but we don't need to provide a link to this as it's not something most people will need to use.

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team

Merge request reports