Skip to content
Snippets Groups Projects

Exclude archived projects from blob search

Merged Siddharth Dungarwal requested to merge 414901-excluded-archived-from-blob-search into master

What does this MR do and why?

Exclude archived projects from blob search

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

  • Make sure advanced search is enabled.
  • Add a commit with some specific message like foobar on 2 projects
  • Archive one of the projects from project settings
  • Now search for foobar. You should see the results from both projects under projects
  • Now enable the feature-flag search_blob_hide_archived_projects
  • Now search again, you should see the results from only non-archived project
  • Repeat the same steps and test for group search as well.

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

Edited by Siddharth Dungarwal

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Siddharth Dungarwal resolved all threads

    resolved all threads

  • added 1 commit

    • 8acf5da6 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Siddharth Dungarwal resolved all threads

    resolved all threads

  • added 1 commit

    • 51dda9bc - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Siddharth Dungarwal resolved all threads

    resolved all threads

  • added 1 commit

    • 3f990eba - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Amparo Luna removed review request for @a_luna

    removed review request for @a_luna

  • mentioned in epic &10957 (closed)

  • added 7 commits

    • f0a8dc43 - Exclude archived projects from blob search
    • a6735ec2 - Add eql? method in found_blob to fix the failing specs
    • 95141e59 - Merge branch 'master' into 414901-excluded-archived-from-blob-search
    • 962b4c1d - Fix rubocop failures after master merge
    • 4b226a3b - Fix pipeline failures
    • 4251c73c - Add spec for eql method
    • b970be01 - Merge branch '414901-excluded-archived-from-blob-search' of...

    Compare with previous version

  • Siddharth Dungarwal requested review from @a_luna

    requested review from @a_luna

  • Amparo Luna approved this merge request

    approved this merge request

  • Amparo Luna removed review request for @a_luna

    removed review request for @a_luna

  • :wave: @a_luna, thanks for approving this merge request.

    This is the first time the merge request is approved. To ensure full test coverage, a new pipeline will be started shortly.

    For more info, please refer to the following links:

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading