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.

Sign up for GitHub

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

Jump to bottom

[tests-only] add tests for public share #9201

Merged
merged 2 commits into from
Nov 22, 2021
Merged

[tests-only] add tests for public share #9201

merged 2 commits into from
Nov 22, 2021

Conversation

swoichha
Copy link
Contributor

@swoichha swoichha commented Nov 9, 2021 •
edited
Loading

Description

add tests for public link

  • Public link to a file and set a password with special characters
  • Public link to a file and disable it after | use the sharing menu to delete the public link
  • Public link to a file and set a password. Modify Password Policies and retry

Related Issue

#8971 (comment)

@CLAassistant
Copy link

CLAassistant commented Nov 9, 2021 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@swoichha swoichha changed the base branch from master to 2.9 November 9, 2021 11:25
@swoichha swoichha self-assigned this Nov 9, 2021
@swoichha swoichha marked this pull request as ready for review November 10, 2021 05:48
@swoichha swoichha marked this pull request as draft November 10, 2021 05:49
@swoichha swoichha marked this pull request as ready for review November 16, 2021 10:15
This was referenced Nov 16, 2021
Closed
Merged
@sonarcloud
Copy link

sonarcloud bot commented Nov 22, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@swoichha swoichha merged commit 842e48d into 2.9 Nov 22, 2021
@delete-merged-branch delete-merged-branch bot deleted the tests-public-share branch November 22, 2021 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@individual-it individual-it individual-it approved these changes

@saw-jan saw-jan Awaiting requested review from saw-jan

Assignees

@swoichha swoichha

Labels
QA:team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@swoichha @CLAassistant @individual-it @saw-jan