Upgrade apollo to version 3.x

[sync]

Why ?

A couple of people from the reasonml community end up with multiple versions of react-apollo in their dependencies and spend time debugging why reason-apollo is no not working for them. (aka: have both react-apollo 2 and react-apollo 3)

So let's use latest Apollo.

TODO

• understand why the swapi example no longer runs

Pull Request Labels

• feature
• blocking
• docs

[sync]

if using yarn temporarily you can force one revision of react-apollo by adding this into your package.json:

"resolutions": {"react-apollo": "3.0.1"}

[MargaretKrutikova]

What error did you get when trying to run swapi? Was it the error about invalid hooks call and three options on what might be causing it?
I got this error trying to run it, and the reason is this:

You might have more than one copy of React in the same app.

I don't think yarn (or npm) handle dependencies properly when developing locally, since the example project is referring to reason-apollo as ../../ in package.json. I found that npm dedupe reason-react after installing node_modules fixes the error.

This problem doesn't appear if you install reason-apollo as usual via yarn add reason-apollo because the dependencies are handled differently in that case.
Let me know if it helps to fix the issue you are having!

[sync]

Completely forgot to reply sorry. So do you think it’s safe the merge. Is it working for you ?

[MargaretKrutikova]

It worked for me when I tested, but it would be nice to get an approval for this from the maintainers 😄

[peterpme]

Bump - can we merge this in @Gregoirevda ?

[peterpme]

Bump - can we please merge this in @Gregoirevda

If you're no longer able to maintain this do you mind having the community help? We're relying on this 😄

[GTDev87]

There are a few performance issues improvements with AC3 that I'm interested in integrating into my project (apollographql/apollo-client#5773). I'd very much like this to be supported.

So bump.