Replace the Prisma GraphQL extension with Apollo’s

[alloy]

The required fix for the extension has been merged, so just waiting for a new release now. apollographql/apollo-tooling#769

[orta]

Yep, I wonder if this will un-install it for folks - will make a release when I get into the office

[alloy]

I doubt it will uninstall. Might be a good idea to send a @-here in the channel that people should uninstall the other 2.

[orta]

Per-repo we can define "unwanted" we might be able to do that here

[jbaxleyiii]

@alloy @orta wanted to let you know that the new extension is being verified on the store right now! Please let us know if it works well for you and any features you'd like to see. I'd love to see it working in a Relay app at some point 😄 and see if the rest of the platform like the schema reg would be valuable for Artsy!

[alloy]

@jbaxleyiii Thanks! Pretty much everything seems to work well thus far (you can try it with this repo) 👍

The only thing that’s not working well right now is in operation variables validation. Relay Modern has the @argumentDefinition directive, which defines ‘local’ variables. These don’t need to be declared at the root of the operation, which currently leads to the many false positives seen below. It would be nice to either have that work or be able to disable that validation rule:

[jbaxleyiii]

Okay great! I’m going to try and play around with this tonight / this weekend. Im particularly interested in making sure operation extraction (and by extension validation) works for all kinds of clients!

I’ve been thinking through adding custom validation rules as part of the config as well for more advanced cases. Thanks for the playground repo 😍