mhallin/graphql_ppx

Validation for (required) query / mutation arguments

anmonteiro opened this issue · 5 comments

The same way this PPX validates if a queried field is not present in the schema, I think it could validate that a required argument isn't passed to a query or a mutation?

I just ran into this and the server threw a runtime error. I think it could be caught at compile time. Is this on the roadmap?

I think it is. It's in the Limitations section of the readme.

Thanks for clarifying, I wasn't sure if this is what that bullet point was referring to.

👋 this issue hasn't had a lot of activity on it but I was wondering if there was a good place to start to try to fix this? Thanks :)

I haven't touched that part of the code but I assume it happens here for BuckleScript and here for Native. It's a really important issue so I hope you will succeed!

I have PR to baransu/graphql_ppx_re implementing such check. If you're interested you can take a look :)