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

Fix ListUserOrgs API #12910

Merged
merged 2 commits into from
Sep 20, 2020
Merged

Fix ListUserOrgs API #12910

merged 2 commits into from
Sep 20, 2020

Conversation

a1012112796
Copy link
Member

fix #12891

fix go-gitea#12891

Signed-off-by: a1012112796 <1012112796@qq.com>
@codecov-commenter
Copy link

codecov-commenter commented Sep 20, 2020

Codecov Report

Merging #12910 into master will increase coverage by 0.00%.
The diff coverage is 3.70%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #12910   +/-   ##
=======================================
  Coverage   43.07%   43.08%           
=======================================
  Files         658      658           
  Lines       72450    72449    -1     
=======================================
+ Hits        31210    31213    +3     
+ Misses      36185    36179    -6     
- Partials     5055     5057    +2     
Impacted Files Coverage Δ
routers/api/v1/misc/signing.go 0.00% <0.00%> (ø)
routers/api/v1/org/label.go 56.71% <0.00%> (ø)
routers/api/v1/repo/commits.go 57.31% <0.00%> (ø)
routers/api/v1/repo/fork.go 17.94% <0.00%> (ø)
routers/api/v1/repo/label.go 56.71% <0.00%> (ø)
routers/api/v1/repo/milestone.go 57.69% <0.00%> (ø)
routers/api/v1/repo/pull.go 25.05% <0.00%> (ø)
routers/api/v1/repo/pull_review.go 35.67% <0.00%> (ø)
routers/api/v1/repo/release.go 33.05% <0.00%> (ø)
routers/api/v1/repo/repo.go 61.68% <0.00%> (ø)
... and 9 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 23f064e...371e2fc. Read the comment docs.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Sep 20, 2020
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Sep 20, 2020
@6543 6543 added modifies/api This PR adds API routes or modifies them type/bug labels Sep 20, 2020
@6543 6543 added this to the 1.13.0 milestone Sep 20, 2020
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Sep 20, 2020
@lafriks lafriks merged commit b74ed16 into go-gitea:master Sep 20, 2020
a1012112796 added a commit to a1012112796/gitea that referenced this pull request Sep 21, 2020
fix go-gitea#12891

Signed-off-by: a1012112796 <1012112796@qq.com>

Co-authored-by: Lauris BH <lauris@nix.lv>
@a1012112796
Copy link
Member Author

Need backport :)

@a1012112796 a1012112796 deleted the fix_nit branch September 21, 2020 04:39
@6543 6543 added backport/done All backports for this PR have been created backport/v1.12 labels Sep 21, 2020
techknowlogick pushed a commit that referenced this pull request Sep 21, 2020
fix #12891

Signed-off-by: a1012112796 <1012112796@qq.com>

Co-authored-by: Lauris BH <lauris@nix.lv>

Co-authored-by: Lauris BH <lauris@nix.lv>
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport/done All backports for this PR have been created lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. modifies/api This PR adds API routes or modifies them type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

API: Get user orgs results in internal server error
5 participants