Skip to content

fix(external-share): move ExternalShareScanJob creation deeper - #64021

Draft
ArtificialOwl wants to merge 1 commit into
masterfrom
fix/noid/move-external-share-scan-deeper
Draft

fix(external-share): move ExternalShareScanJob creation deeper#64021
ArtificialOwl wants to merge 1 commit into
masterfrom
fix/noid/move-external-share-scan-deeper

Conversation

@ArtificialOwl

Copy link
Copy Markdown
Member

This move the creation of the background job ExternalShareScanJob - implemented with #61908 - deeper/closer to the Share Acceptance action.
The reason for this change is that some path to reach the share acceptance (ie. auto-accept on trusted server) would not trigger the background job, therefor not scan the freshly created mountpoint.

@ArtificialOwl ArtificialOwl added this to the Nextcloud 36 milestone Sep 4, 2026
@ArtificialOwl
ArtificialOwl requested a review from a team as a code owner September 4, 2026 12:34
@ArtificialOwl ArtificialOwl added the 3. to review Waiting for reviews label Sep 4, 2026
@ArtificialOwl
ArtificialOwl requested review from Altahrim, CarlSchwan, provokateurin and sorbaugh and removed request for a team September 4, 2026 12:34
@ArtificialOwl

Copy link
Copy Markdown
Member Author

/backport to stable35

@ArtificialOwl
ArtificialOwl marked this pull request as draft September 4, 2026 12:34
@ArtificialOwl

Copy link
Copy Markdown
Member Author

/backport to stable34

@ArtificialOwl

Copy link
Copy Markdown
Member Author

/backport to stable33

@ArtificialOwl ArtificialOwl added 2. developing Work in progress and removed 3. to review Waiting for reviews labels Sep 4, 2026
@ArtificialOwl
ArtificialOwl force-pushed the fix/noid/move-external-share-scan-deeper branch from 8648fdd to 82f050a Compare September 4, 2026 18:14
Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
@ArtificialOwl
ArtificialOwl force-pushed the fix/noid/move-external-share-scan-deeper branch from 82f050a to 5c7f571 Compare September 4, 2026 18:33
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.

1 participant