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

bump reva to latest edge #7727

Merged
merged 3 commits into from Nov 17, 2023
Merged

Conversation

rhafer
Copy link
Contributor

@rhafer rhafer commented Nov 15, 2023

No description provided.

Copy link

update-docs bot commented Nov 15, 2023

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.

@rhafer rhafer marked this pull request as ready for review November 15, 2023 08:33
@rhafer
Copy link
Contributor Author

rhafer commented Nov 15, 2023

cs3org/reva#4343 should fix the failing test

@rhafer
Copy link
Contributor Author

rhafer commented Nov 15, 2023

@micbar @fschade I am not really sure if the current failure is a bug in the code (reva) or in the tests. Is the receiver for a share (with edit role) supposed to be able to see the existing file versions? Then our viewer and editor roles lack the ListFileVersions permission. (Otherwise I somehow think something is borked with the test)

rhafer and others added 3 commits November 16, 2023 14:46
The UnifiedRole related types are pretty specific to the graph service.
Maintaining them as part of reva makes things more complex that required.
Copy link

sonarcloud bot commented Nov 17, 2023

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 2 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@fschade fschade merged commit 50f63bf into owncloud:master Nov 17, 2023
2 checks passed
ownclouders pushed a commit that referenced this pull request Nov 17, 2023
* bump reva to latest edge

To get cs3org/reva#4336

* graph: Import unified role related code from reva

The UnifiedRole related types are pretty specific to the graph service.
Maintaining them as part of reva makes things more complex that required.

* chore: add failing cases to the expected failures

---------

Co-authored-by: Florian Schade <f.schade@icloud.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants