Skip to content

Split code to omit graphql.js bundle duplication in specs

Kos Palchyk requested to merge kpalchyk/fixing_graphql_file_duplication into master

What does this MR do and why?

This is typemaintenance MR that moves TABS value from constants.js to dedicated files. It's needed to split the code in ~/memebers/constants, not to load .vue applications from them. It caused unexpected code duplication in !152975 (merged) pipelines.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

No changes, this is a refactoring.

Group members tab screenshot:

image

How to set up and validate locally

  1. Open group members management page (http://localhost:3000/groups/gitlab-org/-/group_members) and project members management page (http://localhost:3000/gitlab-org/gitlab-test/-/project_members)

  2. Ensure that the pages have tabs with members (depending on the data and settings the instance has, there might be different tabs present). Try inviting a group to a group or project to add "groups" tab.

Edited by Kos Palchyk

Merge request reports