Skip to content

Consider flags enabled when generating GraphQL schema

Brett Walker requested to merge bw-graphql-rake-doc-feature-flag into master

What does this MR do?

!25172 (merged) fixed issue where a developer could generate docs different than what is generated in CI.

With the !26378 (merged), the upgrade to graphql gem 1.9.19 fixes an issue where the visibilty? of a field was not handled correctly. Since we check the feature flag for field visibility, feature flag controlled fields are no longer generated when the schema is output.

Now force enable the feature flags to properly document all fields.

Related to #207278 (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
Edited by Brett Walker

Merge request reports