fetchPolicy no-cache for subscriptions

[oudmane]

I have I case using apollo-client where I don't need caching at all, I set

defaultOptions = {
    query: {
        fetchPolicy: 'no-cache'
    },
        mutate: {
        fetchPolicy: 'no-cache'
    }
} 

I notice .subscribe({query}) doesn't support fetchPolicy option, also tries to cache the result.

I forked and edited apollo-client, I wrapped this

apollo-client/packages/apollo-client/src/core/QueryManager.ts

Lines 918 to 920 in 9bcdafe

	this.dataStore.markSubscriptionResult(
	result,
	transformedDoc,

inside

if (options.fetchPolicy && options.fetchPolicy !== 'no-cache') {
    ...
}

I don't know why is this.dataStore.markSubscriptionResult() is enforced (I can't think of cases this maybe useful), and why isn't there a fetchPolicy for subscriptions.

I'll work on a PR to add fetchPolicy to subscription, please let me know if this isn't needed

Thanks

[hwillson]

This has been addressed in #3773. Thanks for pointing this out!