Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Set mount point target user as storage owner #2872

Merged
merged 1 commit into from
May 10, 2024

Conversation

juliushaertl
Copy link
Member

@juliushaertl juliushaertl commented Mar 19, 2024

Requires nextcloud/server#44294

This will avoid that calling getUserFolder($user)->getByid($id) will fail on paths that have no share permission without a user session.

Fix ONLYOFFICE/onlyoffice-nextcloud#900

@come-nc come-nc force-pushed the fix/set-storage-owner-for-mountpoint branch from 85a17ed to 8869508 Compare April 15, 2024 08:15
@solracsf solracsf added the 3. to review Items that need to be reviewed label Apr 16, 2024
@juliushaertl juliushaertl force-pushed the fix/set-storage-owner-for-mountpoint branch from 8869508 to b80e1c1 Compare April 26, 2024 11:08
Signed-off-by: Julius Härtl <jus@bitgrid.net>
@juliushaertl juliushaertl force-pushed the fix/set-storage-owner-for-mountpoint branch from 774d51c to 91411cb Compare May 8, 2024 20:41
@juliushaertl
Copy link
Member Author

Failure is unrelated (also happens on other prs)

@juliushaertl juliushaertl merged commit 8abd8af into master May 10, 2024
40 of 42 checks passed
@juliushaertl juliushaertl deleted the fix/set-storage-owner-for-mountpoint branch May 10, 2024 10:32
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.

Error with opening documents in Nextcloud Mobile App with Groupfolders Advanced Permissions
4 participants