Skip to content

Remove redundant typename in queries

Miguel Rincon requested to merge 356660-remove-redundant-typename-queries into master

What does this MR do and why?

During our ApolloClient upgrade at !55062 (merged) we changed our mock client configuration to not add the __typename automatically. This forced us to add them manually in our queries to make our jest tests pass.

This change removes the now redundant __typename fields in queries, because now that !95149 (merged) has been merged, the mock client behaves like the production client.

Note: No app/graphql queries in have changed. if we remove it from shared queries, we’ll break graphql-startupjs

Note 2: Today is the code cutoff for %15.3, I propose we wait until %15.3 is closed to avoid merge conflicts in urgent other MRs.

Screenshots or screen recordings

NA

How to set up and validate locally

NA

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

Edited by Miguel Rincon

Merge request reports