Skip to content

Conversation

@provokateurin
Copy link
Member

@provokateurin provokateurin commented Dec 18, 2024

Summary

The behavior was changed in #49694, but didn't account for the fact an empty string is returned instead of the assumed / if the instance is at the root path.

I confirmed this now correctly works with instances hosted at the root path and in a subfolder.

Checklist

@provokateurin provokateurin added bug 3. to review Waiting for reviews labels Dec 18, 2024
@provokateurin provokateurin added this to the Nextcloud 31 milestone Dec 18, 2024
@provokateurin provokateurin requested a review from a team December 18, 2024 15:47
@provokateurin
Copy link
Member Author

/backport to stable30

…ce is hosted at the root path

Signed-off-by: provokateurin <[email protected]>
@Koc
Copy link
Contributor

Koc commented Dec 18, 2024

/compile

@Koc
Copy link
Contributor

Koc commented Dec 18, 2024

@provokateurin thank you for catching and fixing this

Signed-off-by: nextcloud-command <[email protected]>
@sorbaugh sorbaugh disabled auto-merge December 18, 2024 16:19
@sorbaugh sorbaugh merged commit db683da into master Dec 18, 2024
118 of 120 checks passed
@sorbaugh sorbaugh deleted the fix/files/preview-service-worker-registration branch December 18, 2024 16:20
@skjnldsv skjnldsv mentioned this pull request Jan 7, 2025
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 bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants