Skip to content
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

add graphql #994

Closed
wants to merge 8 commits into from
Closed

add graphql #994

wants to merge 8 commits into from

Conversation

anbraten
Copy link
Member

@anbraten anbraten commented Jun 19, 2022

todo

  • map graphql types & fields correctly to models
  • add some basic queries
  • add middlewares / directives
  • add dataloaders
  • add subscription
  • add some docs

@6543 6543 added server feature add new functionality labels Aug 31, 2022
@anbraten anbraten mentioned this pull request Sep 29, 2022
@gapodo
Copy link
Contributor

gapodo commented Nov 6, 2022

I almost feel like necro posting, but I'm currently looking over open PRs / Issues looking for interesting things...

Have you considered some complexity calcs before running queries, as graphql with resolvers is a "nice" way to (intentionally or just by accident) DOS a server with a single request (don't ask me how I know...) or is this baked into the graphql-lib you are using?

@anbraten anbraten mentioned this pull request Nov 26, 2022
31 tasks
@anbraten
Copy link
Member Author

anbraten commented Feb 1, 2023

Although its super anoying that event-streams and websockets have nothing like openapi to be schema defined which could be done by graphql and using graphs for nested structures like pipelines, workflows and steps would be perfect this seems to be a lot of work atm. Maybe we will have a look at it at some point again.

@anbraten anbraten closed this Feb 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature add new functionality server
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants