-
Notifications
You must be signed in to change notification settings - Fork 259
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
wrong type for the body parameter of the grpc directive #2381
Comments
/bounty 100$ |
|
/attempt #2381
|
Thanks @nekinie you are right, I think this is a quick fix. BTW have you checked out our auto-generation guide — https://tailcall.run/docs/graphql-configuration-generation-with-tailcall/ It should reduce your GraphQL onboarding time drastically! |
💡 @ssddOnTop submitted a pull request that claims the bounty. You can visit your bounty board to reward. |
🎉🎈 @ssddOnTop has been awarded $100! 🎈🎊 |
fixed in #2383 |
I gave this a test today and everything looks great, thank you :) |
Prerequisites
Describe the bug
The type for the body parameter of the GRPC directive is string, but the comment around its decleration indicates that static objects should work. I beleve the type should be JSON to support flat objects in the same way that the args parameter for the call directive.
https://github.com/tailcallhq/tailcall/blob/main/generated/.tailcallrc.graphql#L112
The text was updated successfully, but these errors were encountered: