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

[full-ci] Bump Reva Version #5663

Closed
wants to merge 3 commits into from
Closed

Conversation

kobergj
Copy link
Collaborator

@kobergj kobergj commented Feb 27, 2023

bumps reva to include latest bugfixes

@kobergj kobergj force-pushed the BumpRevaVersion branch 2 times, most recently from fe56ecb to bc863d1 Compare February 27, 2023 15:00
Signed-off-by: jkoberg <jkoberg@owncloud.com>
Signed-off-by: jkoberg <jkoberg@owncloud.com>
Signed-off-by: jkoberg <jkoberg@owncloud.com>
@sonarqubecloud
Copy link

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

33.3% 33.3% Coverage
0.0% 0.0% Duplication

@micbar
Copy link
Contributor

micbar commented Feb 28, 2023

seems that we have a hard fail during some uploads

@kobergj
Copy link
Collaborator Author

kobergj commented Mar 1, 2023

Reason why tests are failing: The WebUI tests can't handle async uploads. This is now visible because we now return early on the ocdav GET handler when a file is processing.

We need to adjust them to handle async postprocessing

@butonic
Copy link
Member

butonic commented Mar 7, 2023

@kobergj hm we bumped reva in #5737 ... anything we need to add from this PR? or close?

@kobergj
Copy link
Collaborator Author

kobergj commented Mar 7, 2023

Old and forgotten. Sorry!

@kobergj kobergj closed this Mar 7, 2023
@kobergj kobergj deleted the BumpRevaVersion branch August 8, 2024 09:26
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.

4 participants