-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Add OCS API to send, get, accept and decline remote shares #16733
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
Conversation
|
super cool!!! 👍 |
|
Refer to this link for build results (access rights to CI server needed): |
|
@schiesbn mind having a look at the URL and stuff? Thanks |
|
looks good 👍 |
|
@rullzer are you okay with the direction as well? I want to get this in, as soon as 8.1 is out, so we can have all the smashbox tests with server2server share from the beginning of the 8.2 dev cycle |
|
Yep looking good! 👍 |
|
Rebase of course ;) |
66b1af2 to
2118713
Compare
|
A new inspection was created. |
|
rebased cleanly |
|
Would it be possible to add some tests here? |
|
Well the methods are tested in https://github.com/owncloud/core/blob/master/apps/files_sharing/tests/external/managertest.php ;) |
|
Ah great! Lets merge this baby then, so smashbox can do its thing and break ;) |
Add OCS API to send, get, accept and decline remote shares
|
@nickvergessen Do we want to backport this ? |
|
No, if so we should have merged it before the release.. this is a new feature |
|
Fine by me, makes sense. |
|
Is there any doc for the new apis? |
|
It's OCS, what doc do you need? the URLs or what? If you can point me to other OCS api docs I can prepare a PR |
|
Are there new apis to support the remote shares or is this change an extension of the existing api? |
|
The OCS docs for sharing are here: https://github.com/owncloud/documentation/blob/master/developer_manual/core/ocs-share-api.rst So I guess you could update them ? |
|
@jnfrmarks this just adds a non-UI way for remote shares, so we can use them in smashbox to test federated cloud sharing |
|
Documentation PR at owncloud-archive/documentation#1329 Feel free to review |

This is required so we can manage remote shares via the pyoc client for smashbox
Replace #14668
Fix #13627