Skip to content

[1/3] subgroups #35265

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

Draft
wants to merge 97 commits into
base: main
Choose a base branch
from
Draft

Conversation

GamerGirlandCo
Copy link

@GamerGirlandCo GamerGirlandCo commented Aug 13, 2025

this is the first in a series of 3 pull requests to implement subgroups which closes #1872

checklist

  • models
    • add GroupID field to Repository struct
    • new (to add)
      • Group
      • group-team relation
        stores a team’s access level to a group
      • GroupUnit
        like a repository unit, but grants or denies access to a specific unit at the group level. will override unit permissions defined on a team
  • misc
  • API
    access to these endpoints is governed by read:organization and write:organization scopes
    • POST /org/{org}/groups/new → create a new group at the root level of an org
    • /groups/{group_id}
      • GET → retrieves the group
      • PATCH → updates the group's name and/or description
    • GET /groups/{group_id}/subgroups → get a group's subgroups
    • GET /groups/{group_id}/repos → get a group's repos
    • POST /groups/{group_id}/new → creates a new subgroup inside a group
    • POST /groups/{group_id}/move → moves a group to different group in the same organization, or to the root level if newParent is null or 0
    • POST /{username}/{reponame}/groups/move → moves a repository to a different group within an organization, or to the root level if newParent is null or 0
    • allow a group ID to be specified when creating a repository

/claim #1872

Stack generated by Git-Town

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Aug 13, 2025
@github-actions github-actions bot added the modifies/go Pull requests that update Go code label Aug 13, 2025
@github-actions github-actions bot added the modifies/api This PR adds API routes or modifies them label Aug 13, 2025
@GamerGirlandCo GamerGirlandCo force-pushed the feat/subgroups/base branch 8 times, most recently from c3de504 to 7e4b218 Compare August 14, 2025 21:15
@lunny
Copy link
Member

lunny commented Aug 14, 2025

It seems reusing organization as a sub organization could reduce many duplicated works.

@GamerGirlandCo GamerGirlandCo force-pushed the feat/subgroups/base branch 2 times, most recently from 468974c to c328bda Compare August 16, 2025 23:21
@github-actions github-actions bot added the modifies/cli PR changes something on the CLI, i.e. gitea doctor or gitea admin label Aug 18, 2025
@GamerGirlandCo GamerGirlandCo force-pushed the feat/subgroups/base branch 3 times, most recently from 24457b4 to 7f8cace Compare August 18, 2025 22:17
… bailing out if both ownerID and parentGroupID are < 1
i think they're caused by group permissions causing more repos to be returned than before
add migration for repository table
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🙋 Bounty claim 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 modifies/cli PR changes something on the CLI, i.e. gitea doctor or gitea admin modifies/go Pull requests that update Go code modifies/migrations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Subgroups in Gitea
3 participants