Skip to content

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Aug 5, 2024

Backport of #46767

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Remove all the empty commits

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot backportbot bot requested review from skjnldsv and susnux August 5, 2024 12:09
@backportbot backportbot bot added this to the Nextcloud 28.0.9 milestone Aug 5, 2024
@susnux susnux changed the title [stable28] [stable29] fix(files): Correctly validate new node name [stable28] fix(files): Correctly validate new node name Aug 5, 2024
@susnux susnux force-pushed the backport/46774/stable28 branch from 7f3b516 to c4ec230 Compare August 21, 2024 00:08
@susnux susnux marked this pull request as ready for review August 21, 2024 00:08
@susnux susnux requested a review from nfebe August 21, 2024 00:08
@susnux susnux force-pushed the backport/46774/stable28 branch from c4ec230 to 50a404c Compare August 21, 2024 00:13
@skjnldsv
Copy link
Member

Restarted cypress, but failure looked suspiciously related

@skjnldsv skjnldsv enabled auto-merge August 21, 2024 07:58
@skjnldsv
Copy link
Member

Failure related

@skjnldsv skjnldsv added 2. developing Work in progress and removed 3. to review Waiting for reviews labels Aug 21, 2024
@DaphneMuller
Copy link

@skjnldsv what is the next step here?

@skjnldsv
Copy link
Member

@skjnldsv what is the next step here?

Fixing the tests 🤭🤭

@DaphneMuller
Copy link

@skjnldsv sorry I don't fully get what you mean. The reason I ask is because this is for a support ticket that is meeting the SLA soon. Can this please be merged? Or can you give me a date when it can to be merged? Or otherwise tell me a next step concretely (who does what when)

@susnux
Copy link
Contributor

susnux commented Aug 21, 2024

@DaphneMuller should be in next maintenance releases

@susnux susnux force-pushed the backport/46774/stable28 branch from 50a404c to 27538b5 Compare August 21, 2024 13:37
@susnux
Copy link
Contributor

susnux commented Aug 21, 2024

drone is unrelated

@susnux susnux added 4. to release Ready to be released and/or waiting for tests to finish and removed 2. developing Work in progress labels Aug 21, 2024
@susnux susnux disabled auto-merge August 21, 2024 15:15
susnux added 2 commits August 27, 2024 12:33
* Resolves #45409

This includes two fixes:

1. The name in the "new node" dialog is correctly selected (e.g. `file.txt` only `file` is selected by default), to allow quick naming

2. `@nextcloud/files` functions for filename validation are used, this allows to use new Nextcloud 30 capabilities (e.g. reserved names)

Signed-off-by: Ferdinand Thiessen <[email protected]>
Signed-off-by: Ferdinand Thiessen <[email protected]>
@susnux susnux force-pushed the backport/46774/stable28 branch from 27538b5 to adbffd6 Compare August 27, 2024 10:44
@susnux susnux enabled auto-merge August 27, 2024 10:44
@susnux susnux merged commit 5ecccb9 into stable28 Aug 27, 2024
@susnux susnux deleted the backport/46774/stable28 branch August 27, 2024 12:32
@blizzz blizzz mentioned this pull request Sep 4, 2024
1 task
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

None yet

Development

Successfully merging this pull request may close these issues.

6 participants