-
Notifications
You must be signed in to change notification settings - Fork 187
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
[full-ci] Do not share versions #5531
Conversation
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
E2E tests failed: https://drone.owncloud.com/owncloud/ocis/21170/88/1 💥 To see the trace, please open the link in the console ...
npx playwright show-trace https://cache.owncloud.com/public/owncloud/ocis/21170/tracing/file-alice-2023-4-5-09-54-57.zip |
37c5236
to
56357be
Compare
56357be
to
2cd6257
Compare
d037587
to
ba77d05
Compare
@ScharfViktor Ok, you can take over that PR. Only adapting the acceptance tests is needed. |
let me do a rebase. |
I should probably fix e2e and bring new webBump to PR |
I think I'm done. I bump webCommit and CI should be green. @micbar could you check if the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just small comments. tests look good.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests look good to me. 👍
Kudos, SonarCloud Quality Gate passed! |
* change test expectations * bump reva version * add changelog * fix test * php style fix * bump webCommit. fixt contarct tests * change webBranch * change webBranch * new web commit * fix flaky draw.io file * fix after review --------- Co-authored-by: Viktor Scharf <[email protected]>
Description
Change: Do not share versions
We changed the default behavior of shares: Share receivers have no access to versions. People in spaces with the "Editor" or "Manager" role can still see versions and work with them.
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: