Skip to content
This repository has been archived by the owner on May 26, 2023. It is now read-only.

Integrating with graphql_ppx #15

Closed
thangngoc89 opened this issue Jan 7, 2018 · 3 comments
Closed

Integrating with graphql_ppx #15

thangngoc89 opened this issue Jan 7, 2018 · 3 comments

Comments

@thangngoc89
Copy link

I have tried to integrate reason-apollo with graphql_ppx for a better dev experience like auto linting the query, auto type definition,...

This is what I've done so far https://github.com/thangngoc89/reason-apollo

It's a fork of this repo and maintaining a fork isn't something I want to do. So I hope it could be somehow make it back to this repo

@Gregoirevda
Copy link
Contributor

Thanks for sharing! I'll take a look tomorrow

@emmenko
Copy link
Contributor

emmenko commented Jan 9, 2018

It's a fork of this repo and maintaining a fork isn't something I want to do. So I hope it could be somehow make it back to this repo

We should definitely do this yes!

Maybe can you guys explain a bit the idea and the steps to take for integrating the ppx part? It would be really helpful to understand this a bit more and to help out.

@Gregoirevda
Copy link
Contributor

In PR, Query has the same usage as your example @thangngoc89.
Mutation is added too, would be better to be able to add the parser in the Children as a function 3th argument, but don't think it's possible.

#18

Please leave some comments in the PR before I merge it

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants