vkolgi / json-to-graphql-query

Simple module that converts JavaScript objects to GraphQL query syntax
MIT License
283 stars 39 forks source link

fixing args / directives order according to gql specs #45

Closed MorganDbs closed 2 years ago

MorganDbs commented 2 years ago

Hello,

We noticed an error while using your lib. Your directives should always be declared after your arguments according to gql specs. Find more information here.

Hope you can merge this soon :dancers:

Best regards

vkolgi commented 2 years ago

Thanks for the PR @MorganDbs . That sounds correct at-least from the GraphQL specs. I will merge the PR and release new version.

MorganDbs commented 2 years ago

Thanks you :)

MorganDbs commented 2 years ago

@vkolgi Hello, sorry to be insistent but will you publish the new version soon ?

vkolgi commented 2 years ago

@MorganDbs Sorry about that ! I published the new version now.