Skip to content

Remove GraphQL check for feature flag and return null

Andrew Smith requested to merge espadav8/gitlab:7641-remove-feature-flag-check-from-graphql into master

What does this MR do and why?

When the epic_color_highlight feature flag was added, the GraphQL changes were hidden behind the feature flag. That causes a number of queries to be duplicated in full except for a single field or two.

This changes the GraphQL to instead return null if the feature flag is disabled so that we can use a single query file instead.

See: !89709 (comment 989593822)
See: https://docs.gitlab.com/ee/development/api_graphql_styleguide.html#toggle-the-value-of-a-field

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

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

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

Edited by Andrew Smith

Merge request reports