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

remove leading slash #40843

Merged
merged 1 commit into from
Oct 10, 2023
Merged

remove leading slash #40843

merged 1 commit into from
Oct 10, 2023

Conversation

ArtificialOwl
Copy link
Member

@ArtificialOwl ArtificialOwl commented Oct 9, 2023

follow up of #40794

This will make the redirection works also on nextcloud installed in a subfolder.
Tested on 2 subfolder instances. Can confirm federated sharing works the same pre-ocm-rewrite way

Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
@ArtificialOwl
Copy link
Member Author

(backport will be manual)

@ArtificialOwl ArtificialOwl added this to the Nextcloud 28 milestone Oct 9, 2023
@ArtificialOwl ArtificialOwl requested review from a team, icewind1991, Altahrim and Fenn-CS and removed request for a team October 9, 2023 21:23
@blizzz blizzz added the bug label Oct 10, 2023
@blizzz blizzz enabled auto-merge October 10, 2023 17:24
@blizzz blizzz added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Oct 10, 2023
@blizzz blizzz merged commit a149d4e into master Oct 10, 2023
39 checks passed
@blizzz blizzz deleted the fix/noid/ocm-provider-on-subfolder branch October 10, 2023 17:54
@blizzz
Copy link
Member

blizzz commented Oct 10, 2023

/backport to stable27

@blizzz
Copy link
Member

blizzz commented Oct 10, 2023

the backport to stable27 should work, for stable26 we need to merge #40781

@ArtificialOwl
Copy link
Member Author

/backport to stable26

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants