Skip to content

Reinstate changes from startupJS Graphql with linked queries

John Hope requested to merge jhope-graphql-startup-js-3 into master

What does this MR do?

Reinstates the changes in !36324 (merged)

  • Adds app/grapqhl/queries directory;
  • Changes the startupjs view helper to look there for queries;
  • Add an alias shared_queries for webpack / jest which uses the new location
  • Fails gracefully if no query is found.

!36324 (merged) failed in production because the app/assets directory is removed during build. As a solution we've moved shared queries (i.e. used by both Apollo/Vue and Rails) into their own folder, and aliased this folder in WebPack.

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 John Hope

Merge request reports

Loading