Skip to content

Avoid creating two share links when password is enforced - #25602

Merged
skjnldsv merged 1 commit into
masterfrom
bugfix/noid/only-create-share-once
Feb 22, 2021
Merged

Avoid creating two share links when password is enforced#25602
skjnldsv merged 1 commit into
masterfrom
bugfix/noid/only-create-share-once

Conversation

@juliusknorr

@juliusknorrjuliusknorr commented Feb 12, 2021

Copy link
Copy Markdown
Member

Steps to reproduce:

  • Enforce password on share links
  • Create a new share link and click the -> button next to the password input or click create share

💥 See that the share link was created twice

The main reason is that the close of the modal also triggers the share creation

@skjnldsv
skjnldsv self-requested a review February 19, 2021 10:26
@skjnldsvskjnldsv added 2. developing Work in progress bug labels Feb 19, 2021
@skjnldsv
skjnldsvforce-pushed the bugfix/noid/only-create-share-once branch from 367b63d to c534e8bCompareFebruary 19, 2021 11:02
@skjnldsv
skjnldsv marked this pull request as ready for review February 19, 2021 11:02
@skjnldsvskjnldsv added 3. to review Waiting for reviews feature: sharing and removed 2. developing Work in progress labels Feb 19, 2021
@skjnldsvskjnldsv added this to the Nextcloud 22 milestone Feb 19, 2021
@skjnldsv

Copy link
Copy Markdown
Member

/backport to stable21

@skjnldsv

Copy link
Copy Markdown
Member

/backport to stable20

@skjnldsv

Copy link
Copy Markdown
Member

/backport to stable19

@skjnldsv

Copy link
Copy Markdown
Member

/compile amend /

Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>
@npmbuildbot-nextcloud
npmbuildbot-nextcloudBotforce-pushed the bugfix/noid/only-create-share-once branch from c534e8b to 9073c09CompareFebruary 19, 2021 11:16
@skjnldsv
skjnldsv merged commit 1eca8d2 into masterFeb 22, 2021
@skjnldsv
skjnldsv deleted the bugfix/noid/only-create-share-once branch February 22, 2021 11:08
@skjnldsvskjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Feb 22, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finishbugfeature: sharing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@juliusknorr@skjnldsv@ChristophWurst