Cache redirects

[helfer]

These are the cherry-picked commits from #809 to implement only cache redirects (and not yet fully support client-side computed fields).

TODO:

• If this PR is a new feature, reference an issue where a consensus about the design was reached (not necessary for small changes)
• Make sure all of the significant new logic is covered by tests
• Rebase your changes on master so that they can be merged easily
• Make sure all tests and linter rules pass
• Update CHANGELOG.md with your change
• Add your name and email to the AUTHORS file (optional)
• If this was a change that affects the external API, update the docs and post a link to the PR in the discussion

[helfer]

@stubailo please review this when you have a minute. I think it should work, but a second pair of eyeballs on this would be good.

[stubailo]

I think this can be simplified since the stuff around the ID is mostly boilerplate.

[stubailo]

Hmm, this is probably not the optimal API right? Perhaps we should change this so that people can just return the ID?

[helfer]

Haha, yes, good catch! I patched the tests to get things working and then forgot to build that in.

[stubailo]

Hmm, let's talk about this.

[helfer]

@stubailo I added a toIdValue helper function, which I think is the right way to go, because we want people to indicate if they're returning an id vs. just an object from the resolver.

[stubailo]

because we want people to indicate if they're returning an id vs. just an object from the resolver.

Why?

[helfer]

How would you tell apart an id from a string otherwise? Personally I'm in favor of having as little fancy magic under the hood as possible, and making people use toIdValue seems like not too much of an ask.

[helfer]

I don't know why I already merged this. I still need to update the changelog and write docs for it...

[TSMMark]

Does this fix pagination? I have a component which utilizes pagination, but it fetches data every time the component mounts, even if the data from the exact same query is already cached in redux. Is this a known issue?

Edit: I opened issue #933