-
-
Notifications
You must be signed in to change notification settings - Fork 5
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
Yoga options does not accept schema or yoga property #3
Comments
Example provided in npm registry and this repo readme is outdated from previous version |
@SaltyAom How about sending users to the documentation page instead of putting the same code in the readme.md file of the plugin repo? That way, the documentation would become the source of truth instead of trying to keep it up to date with the documentation and vice versa |
@fecony , got it. Thanks! Regardless, would be nice if this was available, for those that do schema stitching app.use(yoga({ schema })); |
Yes agreed, I wanted to use https://garph.dev/ once but Elysia plugin doesn't support passing schema. Maybe it would be possible to adjust plugin to let us pass schema or what we do now But I didn't try to do it and see how hard it would be, pretty sure @SaltyAom can do it while sleeping xd |
oh wow...that looks enticing. I might try garph out |
Just wake up, make Keep in mind that you will not have type-safety from Mobius if you are using external source. |
Just upgraded to |
Agreed, I'd love to be allowed to pass in our own "schema". For reference, it's because i want to use Pothos (https://github.com/hayes/pothos) |
I'm using schema?: YogaSchemaDefinition<{}> | undefined If there is anything I can do to help. |
any plan for this yet? using |
Pull Request: #8 |
Example from: https://www.npmjs.com/package/@elysiajs/graphql-yoga
Error
Desired, yoga accepts schema property
createSchema()
or frommakeExecutableSchema()
, can only dotypeDefs
andresolvers
separately.The text was updated successfully, but these errors were encountered: