Skip to content

Refactor branch switcher in pipeline editor

Mireya Andres requested to merge pipeline-editor/refactor-branch-switcher into master

What does this MR do?

For #330869 (closed) and #331105 (closed)

This refactor does two things:

  • Addresses the follow up issue where we use fetchMore to implement the pagination for availableBranchesQuery
  • Separates the query into two so that the query for fetching all branches is different from the query for fetching branches using a searchPattern. This is an iterative step to address the bug mentioned in #331105 (comment 581598628) where the proposed solution is to use a network-only fetch policy for searching but use the cache when fetching all branches.

The changes are implemented in two separate commits if the reviewer finds it easier to review by commit 👍

The only real change is the updated fetch policy for searching; there are no new features introduced in this MR, so the existing tests should still pass.

Screenshots

No visual changes to the branch switcher.

Branch Switcher

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Edited by Mireya Andres

Merge request reports