Remove switch to basic search link
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
- [-] Changelog entry
- [-] Documentation (if required)
-
Code review guidelines -
Merge request performance guidelines -
Style guides -
Database guides -
Separation of EE specific content
Availability and Testing
-
Review and add/update tests for this feature/bug. Consider all test levels. See the Test Planning Process. -
Tested in all supported browsers -
Informed Infrastructure department of a default or new setting change, if applicable per definition of done
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