Skip to content

Update user/repos API endpoint description #16503

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

Merged
merged 2 commits into from
Jul 21, 2021
Merged

Conversation

pat-s
Copy link
Member

@pat-s pat-s commented Jul 21, 2021

Currently states

List the repos that the authenticated user owns or has access to

but the endpoint does not list all repos a user has access to, only the ones a user owns

(Also verified and discussed in Discord)

Fixes #16502

@codecov-commenter
Copy link

codecov-commenter commented Jul 21, 2021

Codecov Report

Merging #16503 (8eb4952) into main (28f6f7b) will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main   #16503      +/-   ##
==========================================
+ Coverage   45.50%   45.52%   +0.01%     
==========================================
  Files         718      718              
  Lines       84262    84262              
==========================================
+ Hits        38347    38357      +10     
+ Misses      39751    39744       -7     
+ Partials     6164     6161       -3     
Impacted Files Coverage Δ
routers/api/v1/user/repo.go 41.07% <ø> (ø)
modules/queue/workerpool.go 54.58% <0.00%> (+0.76%) ⬆️
modules/queue/queue_disk_channel.go 72.78% <0.00%> (+1.77%) ⬆️
modules/git/blame.go 68.57% <0.00%> (+2.85%) ⬆️
modules/util/timer.go 85.71% <0.00%> (+42.85%) ⬆️

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 28f6f7b...8eb4952. Read the comment docs.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jul 21, 2021
@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 Jul 21, 2021
@6543 6543 added modifies/api This PR adds API routes or modifies them type/docs This PR mainly updates/creates documentation labels Jul 21, 2021
@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 Jul 21, 2021
@zeripath zeripath merged commit ef39528 into go-gitea:main Jul 21, 2021
AbdulrhmnGhanem pushed a commit to kitspace/gitea that referenced this pull request Aug 10, 2021
Currently states 

> List the repos that the authenticated user owns or has access to

but the endpoint does not list all repos a user has access to, only the ones a user owns

(Also verified and discussed in Discord)

Fixes go-gitea#16502
@go-gitea go-gitea locked and limited conversation to collaborators Oct 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
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/docs This PR mainly updates/creates documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update description of user/repos API endpoint
5 participants