Intended outcome:
Any variables executed in a query should show in the active queries inspector.
Actual outcome:
Variables are always empty for queries. Mutations are fine.
How to reproduce the issue:
{"operationName":"brandSettings","variables":{"id":123123},"query":"query brandSettings($id: Int!) {\n organization(id: $id) {\n id\n orgSettings {\n defaultLocale\n __typename\n }\n brandSettings {\n communityLogo\n footerContent1\n footerContent2\n footerContent3\n primaryColor\n secondaryColor\n localized {\n language\n communityLogo\n footerContent1\n footerContent2\n footerContent3\n __typename\n }\n __typename\n }\n __typename\n }\n}\n"}

network request shows variables":{"id":123123} but Apollo has root: {} 0 keys
Desktop
@afrankel-sfdo I haven't been able to re-create this. Any chance you (or anyone else) could provide a reproduction (tested against 3.0.5)?
@afrankel-sfdo I haven't been able to re-create this. Any chance you (or anyone else) could provide a reproduction (tested against 3.0.5)?
@hwillson is there an easy way to setup a public example apollo client/server app? My test case is private.
Hi! First of all, thanks and good job releasing 3.0.5 @hwillson 馃帀
Regarding this issue with query variables - I'm able to reproduce it too. I'm using the same client setup I used in https://github.com/apollographql/apollo-client-devtools/issues/462 if it gives any help.
I have a similar issue.
https://github.com/apollographql/apollo-client-devtools/issues/564
I also have a similar issue.
@hwillson I think the issue is when we use apollo client v2. See this example here: https://codesandbox.io/s/apollo-client-devtool-variables-94kn5
@hwillson I think the issue is when we use apollo client v2. See this example here: https://codesandbox.io/s/apollo-client-devtool-variables-94kn5
That's also true for us, using v2.
@hwillson can you remove needs-reproducation, we've determined the issues is related to Apollo v2. What else can we do to help move this along?
Thanks @afrankel-sfdo - the absolute best way to help move this along would be if someone is able to take a quick pass at creating a PR to fix this. We'll definitely get to it soon, but a PR would help expedite things for sure. 馃檹