-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
GraphQL for Apiv3 #4182
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
Comments
I am not comfortable to use GraphQL if its not highly necessary. |
I'm generally -0. Last time I talked w/ folks about it, it didn't seem like there was strong support for the server-side in Python, and it would be locking ourselves into a much smaller ecosystem. I think just building a proper REST API with REST Framework is going to be the best path forward for most of our users, and the simplest thing that we can do. |
Maybe worth to mention that Gitlab is moving to GraphQL: https://docs.gitlab.com/ee/api/README.html#road-to-graphql |
I still agree that DRF is least resistance, but GraphQL does solve our issues with schema design, and doesn't put additional burden on us to create a useable schema -- something we haven't been great at designing thus far. With some testing, I'm not worried about the server side implementation. There is graphene_django, the server portion of the graphql implementation for django, it looks mature. I'm the most hesitant about the client side though. I don't think it addresses some of our problems with API client use, and querying looks much less approachable than some of the ORM-y REST clients that we could use. Next steps could be to do a POC, but I'm probably not sold on graphql solving what problems we do have. I'd say we'd be better off enumerating these issues and coming up with patterns that address them instead. The biggest pattern we need to remove is the builder-only modeling, I think we can get this to strictly client side REST if we make some privileged REST models that have a lot more fields in the response. |
I'll say we can close this. I don't think we have enough support to consider graphql as an option here. |
@humitos and I discussed APIv3 more this morning. He brought up GraphQL as an option for maintaining a flexible API client side. I don't have experience using GraphQL, but it could reduce the amount of code we're maintaining, or move the code to a more maintainable spot.
A few points worth considering:
Any other points?
The text was updated successfully, but these errors were encountered: