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

[tests-only] [full-ci] Bump core commit id in reva master #2642

Merged

Conversation

phil-davis
Copy link
Contributor

@phil-davis phil-davis commented Mar 16, 2022

Similar to #2638

Issue owncloud/QA#731

I added the new "product" data item into the capabilities, version and status responses. That will keep those compatible with the new expected responses.

The product name said "owncloud" in some places and "reva" in others. I made them all "reva" so that capabilities and status.php responses will be consistent.

The version in some places was 10.0.9 and in other places 10.0.11 - I made them all 10.0.11 to be consistent. Some day there will be a system for publishing a current version numbering that is not so hard-coded, and can reflect actual versions of this "reva" product rather than a 10.* version number that looks like it came from ownCloud numbering. But for now, at least the numbers are consistent - and that makes the related capabilities and status.php test scenarios pass.

@phil-davis phil-davis self-assigned this Mar 16, 2022
@phil-davis phil-davis force-pushed the bump-core-commit-id-master-20220316 branch 2 times, most recently from 3624083 to 8030bea Compare March 16, 2022 08:23
@phil-davis phil-davis marked this pull request as ready for review March 16, 2022 08:39
@phil-davis phil-davis force-pushed the bump-core-commit-id-master-20220316 branch from 8030bea to 3f0967e Compare March 16, 2022 10:12
@phil-davis phil-davis merged commit 2f8da47 into cs3org:master Mar 16, 2022
@phil-davis phil-davis deleted the bump-core-commit-id-master-20220316 branch March 16, 2022 11:24
Daniel-WWU-IT pushed a commit to Daniel-WWU-IT/reva that referenced this pull request Mar 16, 2022
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.

2 participants