Skip to content

Conversation

@salmart-dev
Copy link
Contributor

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

Summary

Implements IPartialMountProvider in MountProvider for files_external.

This PR depends on #57289 and and #57286 and their dependant PRs to be merged to function correctly.

Once this PR is merged, all providers that do not implement IPartialShareProvider will stop being called during the setup and shares will disappear. It is important this is merged after all share providers implement #57285

TODO

  • Add tests
  • Add indexes for target on oc_share and oc_share_external

Checklist

@salmart-dev salmart-dev added this to the Nextcloud 33 milestone Dec 30, 2025
@salmart-dev salmart-dev self-assigned this Dec 30, 2025
@salmart-dev salmart-dev added the 2. developing Work in progress label Dec 30, 2025
@salmart-dev salmart-dev force-pushed the feature/54562/drop-mounts-on-full-or-provider-setup branch 2 times, most recently from 463f8cd to 5de8e11 Compare December 31, 2025 09:04
@salmart-dev salmart-dev force-pushed the feature/54562/files-sharing-authoritative branch from 8f6d6a4 to 03c02a1 Compare December 31, 2025 09:11
Signed-off-by: Salvatore Martire <4652631+salmart-dev@users.noreply.github.com>
Signed-off-by: Salvatore Martire <4652631+salmart-dev@users.noreply.github.com>
Signed-off-by: Salvatore Martire <4652631+salmart-dev@users.noreply.github.com>
@salmart-dev salmart-dev force-pushed the feature/54562/files-sharing-authoritative branch from 03c02a1 to 63ec2c4 Compare January 2, 2026 10:54
@salmart-dev salmart-dev added 3. to review Waiting for reviews feature: sharing and removed 2. developing Work in progress labels Jan 2, 2026
@salmart-dev salmart-dev marked this pull request as ready for review January 2, 2026 14:54
@salmart-dev salmart-dev requested a review from a team as a code owner January 2, 2026 14:54
@salmart-dev salmart-dev requested review from Altahrim, leftybournes, provokateurin and yemkareems and removed request for a team January 2, 2026 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants