Remove redundant typename in queries
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.
-
I have evaluated the MR acceptance checklist for this MR.
Related to #356660 (closed)