Support FetchPolicy.cacheFirst on Query widget

[lucasavila00]

I'm trying to use the Query widget to query something only if it is not cached but I can't.

If I set the Fetch Policy to cacheFirst I'd like it to be the fetch policy, but since the Query widget replaces the fetch policy to cacheAndNetwork I can't do this.

I don't know why the widget does this but a fix would be to simply remove the code that replaces the fetchPolicy.

[mainawycliffe]

Yea, that line does look unnecessary, not sure why it is there. When the policy is cacheFirst, it should not update stale data, but rather should stop at cache.

[lucasavila00]

Yes, that's the same as I thought.
I was looking at git blame and the commit that introduced it has no fixed issue.
I have been using a forked version without this overriding and I haven't found any issues yet.

Maybe @HofmannZ could shed some light on the problem as this commit was his :)

[mainawycliffe]

@lucasavila00 can you PR this?

[prajwalmr62]

@mainawycliffe I've raised a PR with the changes.

PR 396

[mainawycliffe]

This should be closed by this PR from @micimize

[HofmannZ]

🎉 This issue has been resolved in version 2.1.1-beta.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

[HofmannZ]

🎉 This issue has been resolved in version 3.0.0-beta.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

[HofmannZ]

🎉 This issue has been resolved in version 3.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀