Skip to content

Fix search sorting issue

Dmitry Gruzd requested to merge 255322-fix-search-sorting-issue into master

What does this MR do?

During the testing of !43295 (merged) on staging we noticed that sort doesn't work for all scopes. Turns out that we forgot to pass sort parameter to some classes.

This MR fixes this problem and adds specs to catch similar type of errors in the future.

This feature is not user facing, so I believe we don't need a changelog here.

#255322 (closed)

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

Closes #255322 (closed)

Edited by Dmitry Gruzd

Merge request reports

Loading