Skip to content

Conversation

@salmart-dev
Copy link
Contributor

@salmart-dev salmart-dev commented Dec 29, 2025

Summary

Removes mounts from the MountManager if they were setup by path with partial mount providers (implementing IPartialMountProvider) and a full user setup or provider setup is triggered afterwards.

This prevents setting up mounts multiple times and duplicated mount objects when using partial mount providers in combination with full-provider setup.

TODO

  • Check that the setupForProvider removes the correct mounts
  • Write tests

Checklist

@salmart-dev salmart-dev added this to the Nextcloud 33 milestone Dec 29, 2025
@salmart-dev salmart-dev self-assigned this Dec 29, 2025
@salmart-dev salmart-dev added the 3. to review Waiting for reviews label Dec 29, 2025
@salmart-dev salmart-dev changed the base branch from master to feature/54562/support-partial-share-providers December 29, 2025 19:15
@salmart-dev salmart-dev force-pushed the feature/54562/drop-mounts-on-full-or-provider-setup branch from 9503239 to 239647d Compare December 30, 2025 09:36
@salmart-dev salmart-dev force-pushed the feature/54562/support-partial-share-providers branch from 147446b to a449af6 Compare December 30, 2025 15:43
@salmart-dev salmart-dev force-pushed the feature/54562/drop-mounts-on-full-or-provider-setup branch from 239647d to 463f8cd Compare December 30, 2025 16:46
@salmart-dev salmart-dev changed the base branch from feature/54562/support-partial-share-providers to master December 30, 2025 16:47
@salmart-dev salmart-dev force-pushed the feature/54562/drop-mounts-on-full-or-provider-setup branch from 463f8cd to 5de8e11 Compare December 31, 2025 09:04
@salmart-dev salmart-dev marked this pull request as ready for review January 2, 2026 10:48
@salmart-dev salmart-dev requested a review from a team as a code owner January 2, 2026 10:48
@salmart-dev salmart-dev requested review from Altahrim, leftybournes, provokateurin and yemkareems and removed request for a team January 2, 2026 10:48
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants