[3.3.0] Queries during SSR is always skipped.

[yashok111]

Intended outcome:
Queries should be completed during renderToStringWithData and return HTML with data.

Actual outcome:
We have chained queries, that must wait data from another query. Looks like during rendering on server, all queries's skip option is always true, even if it changed. I have narrow this issue to 3.3.0-rc.3, the #7310 was introduced in it, maybe this is causing the problem. On 3.3.0-rc.2 everything work fine.

How to reproduce the issue:
Sorry, but I don't have time to create repo(. Hope, description above is enough.

Versions:
System:
OS: macOS 10.15.7
Binaries:
Node: 14.13.1 - /usr/local/bin/node
Yarn: 1.22.10 - /usr/local/bin/yarn
npm: 6.14.8 - /usr/local/bin/npm
Browsers:
Chrome: 87.0.4280.67
Edge: 87.0.664.47
Firefox: 82.0.2
Safari: 14.0.1
npmPackages:
@apollo/client: 3.3.1 => 3.3.1
apollo-upload-client: 14.1.3 => 14.1.3
react-apollo-network-status: 5.0.1 => 5.0.1

[benjamn]

@yashok111 Can you show a small example of how you're changing skip to false? I think you're probably right that #7310 triggered this change in behavior, but I'm curious why this.getOptions().skip remains true after you change it.

[yashok111]

Sorry for long response. In this example if tag is not provided query is skipping, and it's causing loading state to be always true on Server and render Placeholder component. If I provide tag everything work fine in this case.

[BowlingX]

Same problem here, when skip is true on the server loading is also true. On the client loading is false. This causes hydration issues as we render e.g. a spinner on the backend but none on the client.