-
Notifications
You must be signed in to change notification settings - Fork 20
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
2.0 Suggestions #113
Comments
Merge graphql to master! It's (1) very useful for REST applications for easier debugging and things like small database migrations and (2) allows for GraphQL applications. Doesn't break much. Might need some reworking, I can take a look into that ;) There was that Shields problem we discussed though. I could create a onGraphQL or try to assign all actions to the proper shields... |
Rethink Hooks... I believe some hooks (if not all) are now obsolete due to Shields. We should rethink all of them carefully to make sure no unneeded API goes through 2.0, but still guaranteeing all current possibilities. |
Some things that come to mind after having looked at the guides from the website:
|
|
Thanks a lot for the detailed answers! So,
|
Hi! We're starting a new major version of the API. If you have any suggestions about the API or any kind of cleanup, feel free to add as comment.
The text was updated successfully, but these errors were encountered: