Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[stable28] fix(SharingEntryLink): Show default password before create if any #49148

Merged
merged 3 commits into from
Nov 9, 2024

Conversation

backportbot[bot]
Copy link

@backportbot backportbot bot commented Nov 8, 2024

Backport of #49115

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.

@backportbot backportbot bot requested review from skjnldsv, susnux and nfebe November 8, 2024 10:22
@backportbot backportbot bot added this to the Nextcloud 28.0.13 milestone Nov 8, 2024
@nfebe nfebe force-pushed the backport/49115/stable28 branch from d273f32 to 145d1fe Compare November 8, 2024 16:09
@nfebe nfebe marked this pull request as ready for review November 8, 2024 16:10
@nfebe nfebe enabled auto-merge November 8, 2024 16:10
nfebe and others added 2 commits November 8, 2024 19:03
Prevent silent addition of expiration date to shares.

Prevent silent addition for password to shares.

Both now have a `shareRequiresReview` check

Resolves : #48860

Signed-off-by: nfebe <fenn25.fn@gmail.com>
Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
@nfebe nfebe force-pushed the backport/49115/stable28 branch from 145d1fe to b3d0c54 Compare November 8, 2024 18:03
@nfebe
Copy link
Contributor

nfebe commented Nov 8, 2024

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@nfebe nfebe merged commit 4692f8d into stable28 Nov 9, 2024
47 checks passed
@nfebe nfebe deleted the backport/49115/stable28 branch November 9, 2024 13:24
@skjnldsv skjnldsv mentioned this pull request Nov 27, 2024
4 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants