Skip to content

Conversation

@susnux
Copy link
Contributor

@susnux susnux commented Sep 12, 2025

Summary

newPassword is the unsaved password, while share.password is the current saved password.

Checklist

@susnux susnux added this to the Nextcloud 31.0.10 milestone Sep 12, 2025
@susnux susnux requested review from a team, Antreesy, nfebe, salmart-dev, sorbaugh and szaimen and removed request for a team September 12, 2025 10:04
@susnux
Copy link
Contributor Author

susnux commented Sep 12, 2025

/backport to stable32

@susnux
Copy link
Contributor Author

susnux commented Sep 12, 2025

/backport to master

Copy link
Contributor

@szaimen szaimen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🐘

Copy link
Contributor

@Antreesy Antreesy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested, looks good code-wise
Image

`newPassword` is the unsaved password, while `share.password` is the
current saved password.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux force-pushed the fix/files-sharing-password branch from e95fadb to ee97330 Compare September 12, 2025 10:13
@susnux
Copy link
Contributor Author

susnux commented Sep 12, 2025

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@mpivchev
Copy link

Is this patchable? Since it's vue and frontend? I am not sure

@susnux
Copy link
Contributor Author

susnux commented Sep 24, 2025

@mpivchev not really, best to wait for the release on October 9, 2025

@blizzz blizzz mentioned this pull request Oct 14, 2025
Sign up for free to 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.

6 participants