Support Relay v2.0.0

[johntran]

Hi there,

At the time of writing this issue, Relay is on 1.7.0-rc.1, so it is a little early to fix.

I attempted to load up Relay 1.7.0-rc.1 with found-relay, and it is erroring here:

https://github.com/facebook/relay/blob/98e866f5e0fc3548f0577bf081df0507832b15f9/packages/react-relay/modern/buildReactRelayContainer.js#L80

with an error of .unstable_read is not a function

I think its because found-relay expects the legacy Context API, while Relay is using the new Context in React 16.

Reference to Context.unstable_read: facebook/react#13139

Any thoughts?

Though it is possible that its not a problem of found-relay, but its just that the Relay master branch is unstable. I thought it would be related to found-relay because its erroring on components that are used as routes.

[taion]

Yeah, I think we'll need to release a breaking change here to move to the new context API, if Relay 1.7.0 does.

[GrigoryPtashko]

Are there any plans on Relay 2.0.0 support? I want to upgrade my app to the newest version of Relay (I'm using Found Relay).

[taion]

It's not out yet, is it? Are you on the RC?

[taion]

RC 1 doesn't expose the context object, so it wouldn't work. We need to wait for facebook/relay#2547 to land and for the next RC to drop.

[GrigoryPtashko]

Yeah, I'm on 2.0.0-RC1 in my mobile app. But in the webapp I'm on 1.4. I want to upgrade 1.4 to 2.0.

[wkerswell-gresham]

V2 was released 2 days ago so probably safe to upgrade now. Is there plans to update this soon?

[taion]

nice, i'll work on upgrading

[xdissent]

I've been using this branch for a few days and it seems to do the job: https://github.com/xdissent/found-relay/tree/relay-2 Would you like me to PR to get the ball rolling? It's probably a good time to clean up old stuff (getRequest/getOperation comes to mind) but I didn't go that far.

[taion]

oh, that would be great – please do!

[taion]

resolved in #167