Skip to content

[stable30] fix(files_sharing): Password field must not be required if already set - #49013

Merged
susnux merged 2 commits into
stable30from
backport/48991/stable30
Oct 30, 2024
Merged

[stable30] fix(files_sharing): Password field must not be required if already set#49013
susnux merged 2 commits into
stable30from
backport/48991/stable30

Conversation

@backportbot

@backportbotbackportbotBot commented Oct 30, 2024

Copy link
Copy Markdown

Backport of #48991

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

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

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

If there is already a password, there is no need to require the password
in the setting ('newPassword'). It is only required for new shares.
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux
susnuxforce-pushed the backport/48991/stable30 branch from 8446118 to 5f157d1CompareOctober 30, 2024 15:45
@susnux
susnux marked this pull request as ready for review October 30, 2024 15:45
@susnuxsusnux mentioned this pull request Oct 30, 2024
2 tasks
@susnux

Copy link
Copy Markdown
Contributor

/compile

@susnux
susnux enabled auto-merge October 30, 2024 15:57
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@susnux
susnux merged commit 2ebef1e into stable30Oct 30, 2024
@susnux
susnux deleted the backport/48991/stable30 branch October 30, 2024 16:56
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@susnux@provokateurin@nextcloud-command