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

[stable24] Allow compatibility wrapper on local external storage #32273

Merged
merged 1 commit into from
Aug 4, 2022

Conversation

backportbot-nextcloud[bot]
Copy link

backport of #32152

The check was likely in place to prevent adding the wrapper on the root
and home storage, which is not possible anyway since the encoding option cannot
be set on that mount.

Signed-off-by: Vincent Petry <[email protected]>
@backportbot-nextcloud backportbot-nextcloud bot added this to the Nextcloud 24.0.1 milestone May 5, 2022
@blizzz blizzz added the 3. to review Waiting for reviews label May 16, 2022
@blizzz blizzz mentioned this pull request May 16, 2022
@blizzz
Copy link
Member

blizzz commented May 16, 2022

missing reviewers, moving to 24.0.2

@blizzz blizzz mentioned this pull request Jun 9, 2022
2 tasks
@blizzz
Copy link
Member

blizzz commented Jun 13, 2022

moving to 24.0.3…

@blizzz blizzz mentioned this pull request Jun 17, 2022
This was referenced Jul 5, 2022
@blizzz blizzz mentioned this pull request Jul 18, 2022
@blizzz
Copy link
Member

blizzz commented Jul 18, 2022

and moving to 24.0.4… @PVince81 please close the PR if it is not necessary, otherwise please find some reviewers.

@blizzz blizzz mentioned this pull request Aug 1, 2022
Copy link
Member

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@blizzz
Copy link
Member

blizzz commented Aug 1, 2022

@PVince81 this moved pushed already three times. please find reviewers or close this PR if it is not necessary.

@PVince81 PVince81 requested a review from szaimen August 1, 2022 13:06
@PVince81
Copy link
Member

PVince81 commented Aug 1, 2022

@artonge @CarlSchwan @icewind1991 second review please

@blizzz
Copy link
Member

blizzz commented Aug 4, 2022

Samba test is failing, but might be a hiccup:

Screenshot_20220804_111142

Restarting the drone job.

@blizzz
Copy link
Member

blizzz commented Aug 4, 2022

Samba is green => merging

@blizzz blizzz merged commit d81e38d into stable24 Aug 4, 2022
@blizzz blizzz deleted the backport/32152/stable24 branch August 4, 2022 09:42
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.

3 participants