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(files): add missing Overwrite: F header in renaming #41331

Merged
merged 1 commit into from
Nov 8, 2023

Conversation

skjnldsv
Copy link
Member

@skjnldsv skjnldsv commented Nov 7, 2023

Fix #40906

While we have some frontend sanity checks, it doesn't hurt to be sure

@skjnldsv skjnldsv added this to the Nextcloud 28 milestone Nov 7, 2023
@skjnldsv skjnldsv requested review from Altahrim and a team November 7, 2023 16:02
@skjnldsv skjnldsv self-assigned this Nov 7, 2023
@skjnldsv skjnldsv requested review from artonge, nfebe and sorbaugh and removed request for a team November 7, 2023 16:02
@skjnldsv skjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Nov 7, 2023
@skjnldsv skjnldsv force-pushed the fix/files/overwrite-header branch 2 times, most recently from 9db6dd8 to 7764763 Compare November 7, 2023 16:45
Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
@skjnldsv skjnldsv force-pushed the fix/files/overwrite-header branch from 7764763 to f434483 Compare November 8, 2023 08:11
@skjnldsv skjnldsv merged commit 9285fe0 into master Nov 8, 2023
38 of 39 checks passed
@skjnldsv skjnldsv deleted the fix/files/overwrite-header branch November 8, 2023 08:26
@blizzz blizzz mentioned this pull request Nov 8, 2023
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 feature: files
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[Bug]: Overwrite header is missing, leading to deleted file
3 participants