Skip to content

Conversation

qwerty287
Copy link
Contributor

Add an API endpoint which checks if a team has repository access.
Closes #19452

@Gusted Gusted added this to the 1.17.0 milestone Apr 28, 2022
@Gusted Gusted added type/feature Completely new functionality. Can only be merged if feature freeze is not active. modifies/api This PR adds API routes or modifies them labels Apr 28, 2022
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Apr 28, 2022
@Gusted
Copy link
Contributor

Gusted commented Apr 28, 2022

PR to add test-case: https://github.com/qwerty287/gitea/pull/3 otherwise L G T M

@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 Apr 28, 2022
@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 Apr 30, 2022
@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (main@edff571). Click here to learn what that means.
The diff coverage is 75.55%.

@@           Coverage Diff           @@
##             main   #19540   +/-   ##
=======================================
  Coverage        ?   47.39%           
=======================================
  Files           ?      952           
  Lines           ?   132620           
  Branches        ?        0           
=======================================
  Hits            ?    62855           
  Misses          ?    62187           
  Partials        ?     7578           
Impacted Files Coverage Δ
routers/api/v1/org/team.go 48.69% <74.41%> (ø)
routers/api/v1/api.go 76.11% <100.00%> (ø)

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 edff571...63168b0. Read the comment docs.

@6543 6543 merged commit ac6c338 into go-gitea:main May 1, 2022
zjjhot added a commit to zjjhot/gitea that referenced this pull request May 3, 2022
* giteaofficial/main:
  Fix some slice problems (incorrect slice length) (go-gitea#19592)
  Fix sending empty notifications (go-gitea#19589)
  Handle the error of a missing blob object fix go-gitea#19530 (go-gitea#19552)
  Remove legacy `+build:` constraint (go-gitea#19582)
  Federation: return useful statistic information for nodeinfo (go-gitea#19561)
  Upgrade required git version to 2.0 (go-gitea#19577)
  add smtp password to install page (go-gitea#17564)
  ignore DNS error when doing migration allow/block check (go-gitea#19566)
  [skip ci] Updated translations via Crowdin
  Dont overwrite err with nil & rename PullCheckingFuncs to reflect there usage (go-gitea#19572)
  Improve UI on mobile (go-gitea#19546)
  Add API to check if team has repo access (go-gitea#19540)
@qwerty287 qwerty287 deleted the check-team-repo branch May 20, 2022 06:48
AbdulrhmnGhanem pushed a commit to kitspace/gitea that referenced this pull request Aug 24, 2022
* Add API to check if team has repo access

* Add test case
@go-gitea go-gitea locked and limited conversation to collaborators May 3, 2023
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/feature Completely new functionality. Can only be merged if feature freeze is not active.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[API] check if repository is added to team
7 participants