Skip to content

Conversation

@backportbot-nextcloud
Copy link

backport of #25768

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.

👍

@skjnldsv skjnldsv mentioned this pull request Jan 6, 2023
5 tasks
@skjnldsv
Copy link
Member

skjnldsv commented Jan 6, 2023

/rebase

@skjnldsv
Copy link
Member

skjnldsv commented Jan 6, 2023

Lots of failures

@skjnldsv skjnldsv added the 3. to review Waiting for reviews label Jan 6, 2023
@LukasReschke LukasReschke removed their request for review January 6, 2023 17:15
This was referenced Jan 11, 2023
@blizzz blizzz mentioned this pull request Feb 6, 2023
@blizzz
Copy link
Member

blizzz commented Feb 7, 2023

/rebase

the method is only called if the file exists already

a check against storing the fallback mimetype is added as a safety instead

Signed-off-by: Robin Appelman <[email protected]>
@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 Feb 7, 2023
@blizzz blizzz enabled auto-merge February 7, 2023 17:45
@blizzz blizzz disabled auto-merge February 9, 2023 13:35
@blizzz blizzz merged commit 6a19833 into stable25 Feb 9, 2023
@blizzz blizzz deleted the backport/25768/stable25 branch February 9, 2023 13:36
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 technical debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants