-
Notifications
You must be signed in to change notification settings - Fork 16
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
Get rid of t3n/graphql
dependency
#120
Comments
Sounds great. In the beginning we thought introducing a GraphQL package to the core would also increase usage and more 3rd party GraphQL apis via community packages. We should still keep that in mind a bit. |
Another point is that the media-ui cannot be tested and used with PHP 8 because |
Core GraphQL sounds amazing, but is also not easy to maintain i can imagine |
From Slack:
|
The "graphqlite.thecodingmachine.io"-approach didn't really work out for me because:
My current idea would be a package that generates PHP classes (i.e. value objects) for a given schema file and the minimal glue code. This should be fairly simple(tm) in general. But it would be nice of course if the generated code could be edited and re-generated without losing changes (maybe with the help of https://getrector.org/) I currently don't get around working on that, but I didn't forget about this issue. |
FYI: In the meantime I have completely rewritten Wwwision.GraphQL It follows a different approach than described above by going a PHP-first approach, but it might be a good fit anyways. |
I just opened t3n/graphql-tools#49 when I realized the transitive dependency on webonyx/graphql-php is horribly outdated ( |
I actually wanted to check how much effort it would be to switch from GraphQL to REST again. The PHP side should be manageable, but in the JS part I would have to check how well the Apollo RestAdapter works. |
The
t3n/graphql
package offers some great tools, especially to get started. But it has some slight drawbacks:webonyx/graphql-php
~0.13
vs14.11.5
, viat3n/graphql-tools
)For the Media UI package it should be enough to create a really slim layer over (a recent version of)
webonyx/graphql-php
The text was updated successfully, but these errors were encountered: