Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[stable26] Show pending popover menu when password is enabled by default #38396

Merged
merged 3 commits into from Jun 13, 2023

Conversation

backportbot-nextcloud[bot]
Copy link

@backportbot-nextcloud backportbot-nextcloud bot commented May 22, 2023

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

backport of #38164

@nickvergessen
Copy link
Member

/compile /

@nickvergessen nickvergessen added the 3. to review Waiting for reviews label May 23, 2023
@blizzz blizzz mentioned this pull request May 24, 2023
@blizzz blizzz mentioned this pull request Jun 12, 2023
@blizzz
Copy link
Member

blizzz commented Jun 13, 2023

/compile /

@szaimen
Copy link
Contributor

szaimen commented Jun 13, 2023

conflicts

@szaimen szaimen added 2. developing Work in progress and removed 3. to review Waiting for reviews labels Jun 13, 2023
@blizzz
Copy link
Member

blizzz commented Jun 13, 2023

@nextcloud/server-frontend could someone take this over since @danxuliu is OOTO?

danxuliu and others added 3 commits June 13, 2023 15:08
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
(cherry picked from commit d80f8fc)
When "Enforce password protection" is enabled in the sharing settings a
popover menu is shown to set a password before the share is created. On
the other hand, when "Always ask for a password" was enabled in the
sharing settings and a new link share was created the share was
immediately created with a default password; the user was not able to
specify a password (nor create the share without password).

The component template already provided the needed elements to also ask
for the password without enforcing it, but the popover menu was not
shown due to "enableLinkPasswordByDefault" being missing in "if"
conditions.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
(cherry picked from commit d463192)
Signed-off-by: Joas Schilling <coding@schilljs.com>
(cherry picked from commit 9f8fa48)
@Pytal Pytal added 4. to release Ready to be released and/or waiting for tests to finish and removed 2. developing Work in progress labels Jun 13, 2023
@szaimen szaimen enabled auto-merge June 13, 2023 22:43
@szaimen szaimen merged commit f8655c1 into stable26 Jun 13, 2023
38 checks passed
@szaimen szaimen deleted the backport/38164/stable26 branch June 13, 2023 22:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish bug feature: sharing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants