Skip to content

WIP: init graphql support #14721

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

Closed
wants to merge 1 commit into from
Closed

Conversation

a1012112796
Copy link
Member

not like #12285 use gqlgen as frame

add two operations as init
query: viewer
mutation: createRepository

use [gqlgen](https://github.com/99designs/gqlgen) as frame

add two operations as init
query: viewer
mutation: createRepository

Signed-off-by: a1012112796 <[email protected]>
@a1012112796 a1012112796 added modifies/api This PR adds API routes or modifies them pr/wip This PR is not ready for review labels Feb 18, 2021
@SirMishaa
Copy link

Up ? Do you need some help for this PR ?

I'm waiting a lot the support for GraphQL.
It is game changer.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Apr 3, 2021
.PHONY: generate-graphql
generate-graphql:
cd modules/graphql && \
mod=vendor $(GO) run github.com/99designs/gqlgen genrate && \
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't there be generate?

@@ -29,7 +30,7 @@ require (
github.com/ethantkoenig/rupture v1.0.0
github.com/gliderlabs/ssh v0.3.1
github.com/glycerine/go-unsnap-stream v0.0.0-20190901134440-81cf024a9e0a // indirect
github.com/go-chi/chi v1.5.1
github.com/go-chi/chi v3.3.2+incompatible
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems incorrect, it would be ok to upgrade to v4

@wxiaoguang
Copy link
Contributor

Is it feasible for current Gitea code base?

@wxiaoguang wxiaoguang added the issue/needs-feedback For bugs, we need more details. For features, the feature must be described in more detail label May 1, 2023
@a1012112796 a1012112796 closed this May 1, 2023
@a1012112796 a1012112796 deleted the init_graphql branch May 1, 2023 06:22
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Aug 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
issue/needs-feedback For bugs, we need more details. For features, the feature must be described in more detail lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. modifies/api This PR adds API routes or modifies them pr/wip This PR is not ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants