Skip to content

feat(Teams): Add quota management to TeamFolder and ITeamFolderProvider - #63651

Open
Fin-c wants to merge 2 commits into
nextcloud:masterfrom
Dataport:feature/moveTeamFolder_into_adminsettings
Open

feat(Teams): Add quota management to TeamFolder and ITeamFolderProvider#63651
Fin-c wants to merge 2 commits into
nextcloud:masterfrom
Dataport:feature/moveTeamFolder_into_adminsettings

Conversation

@Fin-c

@Fin-c Fin-c commented Aug 26, 2026

Copy link
Copy Markdown

Summary

Add storage quota support to the TeamFolder public API.

Changes

  • Add an optional quota property to TeamFolder
  • Expose the quota through getQuota()
  • Include the quota in the serialized TeamFolder response
  • Extend ITeamFolderProvider with methods to create and update team folder quotas
  • Represent quotas in bytes, with 0 meaning unlimited
  • Use null when the active provider does not expose quota information

TODO

  • ...

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Signed-off-by: Fin Clausen <Fin.Clausen@dataport.de>
@Fin-c Fin-c added this to the Nextcloud 35 milestone Aug 26, 2026
@Fin-c
Fin-c requested a review from a team as a code owner August 26, 2026 08:34
@Fin-c
Fin-c requested review from come-nc, leftybournes, provokateurin and salmart-dev and removed request for a team August 26, 2026 08:34
@Fin-c Fin-c added the 3. to review Waiting for reviews label Aug 26, 2026
@github-project-automation github-project-automation Bot moved this to Backlog in 👥 Teams Aug 26, 2026
@Fin-c
Fin-c force-pushed the feature/moveTeamFolder_into_adminsettings branch from aebfcb7 to be4d00c Compare August 28, 2026 07:54
Co-authored-by: Copilot <copilot@github.com>
Assisted-by: GitHub Copilot:GPT-5.6 Terra
Signed-off-by: Fin Clausen <Fin.Clausen@dataport.de>
@Fin-c
Fin-c force-pushed the feature/moveTeamFolder_into_adminsettings branch from be4d00c to 328eb92 Compare August 28, 2026 08:03
@artonge
artonge requested a review from cristianscheid August 28, 2026 08:16
@Fin-c
Fin-c enabled auto-merge August 28, 2026 09:27
@alimmroth alimmroth moved this from Backlog to In progress in 👥 Teams Aug 28, 2026
@alimmroth alimmroth moved this from In progress to In review in 👥 Teams Aug 28, 2026
@joshtrichards joshtrichards added enhancement feature: teams Related to Teams or circles labels Aug 29, 2026

@cristianscheid cristianscheid left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

just a minor docblock issue, other than that, looks good to me

}

/**
* @return array{id: int, mountPoint: string, quota: int|null}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
* @return array{id: int, mountPoint: string, quota: int|null}
* @return array{id: int, quota: int|null, mountPoint: string}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews enhancement feature: teams Related to Teams or circles

Projects

Status: In review

Development

Successfully merging this pull request may close these issues.

Teams Adminsettings: Show a List of Teams and assigned Team-Folders

5 participants