Skip to content

[stable19] Only clear share password model when actually saved - #26060

Merged
ChristophWurst merged 1 commit into
stable19from
backport/25911/stable19
Mar 15, 2021
Merged

[stable19] Only clear share password model when actually saved#26060
ChristophWurst merged 1 commit into
stable19from
backport/25911/stable19

Conversation

@backportbot-nextcloud

@backportbot-nextcloudbackportbot-nextcloudBot commented Mar 11, 2021

Copy link
Copy Markdown

⚠️ This backport had conflicts and is incomplete ⚠️

backport of #25911

@PVince81

Copy link
Copy Markdown
Member

/compile amend /

@PVince81PVince81 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

@PVince81PVince81 self-assigned this Mar 11, 2021
@PVince81

Copy link
Copy Markdown
Member

going the manual route... will rebuild this

Instead of always clearing the password in the share model, now only do
it when it was actually saved.
Fixes an issue where saving another field would clear the password too
early and prevent it to be saved when the dropdown closed.
Signed-off-by: Vincent Petry <vincent@nextcloud.com>
Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>
@PVince81
PVince81force-pushed the backport/25911/stable19 branch from 593852f to 20c16e2CompareMarch 12, 2021 11:24
@ChristophWurst
ChristophWurst merged commit 3ac0601 into stable19Mar 15, 2021
@ChristophWurst
ChristophWurst deleted the backport/25911/stable19 branch March 15, 2021 08:48
@rullzerrullzer mentioned this pull request Mar 26, 2021
@rullzerrullzer mentioned this pull request Apr 8, 2021
1 task
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.

4 participants

@PVince81@rullzer@skjnldsv@ChristophWurst