graphql-parser icon indicating copy to clipboard operation
graphql-parser copied to clipboard

Published 20 hours ago • verified publisher icon graphql-rust

Add VARIABLE_DEFINITION directive location

Open vmagro opened this issue 2 years ago • 3 comments

I have some schemas that have this directive location, and this crate currently crashes while trying to parse it.

vmagro avatar Jun 27 '22 13:06 vmagro

Perhaps this and https://github.com/graphql-rust/graphql-parser/pull/61 can be merged

LegNeato avatar Aug 04 '22 15:08 LegNeato

@LegNeato I added a test that is similar to other directive parsers, let me know if you had something else in mind

vmagro avatar Aug 31 '22 21:08 vmagro

@LegNeato once this lands, any chance we could get a new version of this crate pushed to crates.io along with a new version of graphql-client to pick up https://github.com/graphql-rust/graphql-client/pull/430?

bolinfest avatar Aug 31 '22 21:08 bolinfest

@LegNeato I think this PR was never included in a release? any chance we can get a release for that fix?

dotansimha avatar Feb 15 '24 08:02 dotansimha