Skip to content

Conversation

@icewind1991
Copy link
Member

@icewind1991 icewind1991 commented Jun 19, 2025

Removes the assumption that all groupfolders are on the same storage. Instead we track the storage id and root id with the folder.

Currently all groupfolders will still be on the root storage, but this opens up the option to have them on different storages down the line.

Included is a cleanup of moving the folder data from an array to a proper type to make it fully clear which api return what fields.

@icewind1991 icewind1991 force-pushed the per-groupfolder-storage branch 8 times, most recently from 642cb1c to c3d0dc4 Compare June 27, 2025 13:28
@icewind1991 icewind1991 changed the title [WIP] Per-groupfolder storage Allow per-groupfolder storage Jun 27, 2025
@icewind1991 icewind1991 added the 3. to review Items that need to be reviewed label Jun 27, 2025
@icewind1991 icewind1991 added this to the Nextcloud 32 milestone Jun 27, 2025
@icewind1991 icewind1991 force-pushed the per-groupfolder-storage branch 4 times, most recently from 6064534 to 6c99421 Compare June 27, 2025 16:06
@icewind1991 icewind1991 marked this pull request as ready for review June 27, 2025 16:12
@icewind1991 icewind1991 force-pushed the per-groupfolder-storage branch from 6c99421 to 3da6c9d Compare July 1, 2025 16:11
@susnux susnux requested review from a team and nfebe and removed request for a team and artonge July 1, 2025 17:18
@icewind1991 icewind1991 force-pushed the per-groupfolder-storage branch 2 times, most recently from 985d5ed to 35cc3a3 Compare July 7, 2025 11:41
Signed-off-by: Robin Appelman <[email protected]>
@icewind1991 icewind1991 force-pushed the per-groupfolder-storage branch from 35cc3a3 to bb5862e Compare July 7, 2025 11:51
@icewind1991 icewind1991 requested a review from come-nc July 7, 2025 11:57
@icewind1991 icewind1991 merged commit eec119e into master Jul 8, 2025
50 checks passed
@icewind1991 icewind1991 deleted the per-groupfolder-storage branch July 8, 2025 12:40
@skjnldsv skjnldsv mentioned this pull request Aug 20, 2025
@icewind1991 icewind1991 mentioned this pull request Aug 27, 2025
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Items that need to be reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants