Skip to content

Conversation

@nickvergessen
Copy link
Contributor

This is required so we can manage remote shares via the pyoc client for smashbox

Replace #14668
Fix #13627

@karlitschek
Copy link
Contributor

super cool!!! 👍

@ghost
Copy link

ghost commented Jun 3, 2015

Refer to this link for build results (access rights to CI server needed):
https://ci.owncloud.org//job/pull-request-analyser-ng-simple/12687/
🚀 Test PASSed.🚀
chuck

@nickvergessen
Copy link
Contributor Author

@schiesbn mind having a look at the URL and stuff? Thanks

@schiessle
Copy link
Contributor

looks good 👍

@nickvergessen
Copy link
Contributor Author

@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

@rullzer
Copy link
Contributor

rullzer commented Jun 30, 2015

Yep looking good! 👍

@rullzer
Copy link
Contributor

rullzer commented Jun 30, 2015

Rebase of course ;)

@nickvergessen nickvergessen force-pushed the add-smashbox-tests-remote-shares branch from 66b1af2 to 2118713 Compare July 3, 2015 07:31
@scrutinizer-notifier
Copy link

A new inspection was created.

@nickvergessen
Copy link
Contributor Author

rebased cleanly
no idea what was wrong that it couldn't be merged before.

@ghost
Copy link

ghost commented Jul 3, 2015

🚀 Test PASSed.🚀
chuck

@rullzer
Copy link
Contributor

rullzer commented Jul 4, 2015

Would it be possible to add some tests here?
Maybe just inject some stuff in the database and see if this retrieves it clearly etc..

@nickvergessen
Copy link
Contributor Author

@rullzer
Copy link
Contributor

rullzer commented Jul 4, 2015

Ah great! Lets merge this baby then, so smashbox can do its thing and break ;)

nickvergessen added a commit that referenced this pull request Jul 6, 2015
Add OCS API to send, get, accept and decline remote shares
@nickvergessen nickvergessen merged commit fca6bcb into master Jul 6, 2015
@nickvergessen nickvergessen deleted the add-smashbox-tests-remote-shares branch July 6, 2015 08:12
@PVince81
Copy link
Contributor

PVince81 commented Jul 6, 2015

@nickvergessen Do we want to backport this ?

@nickvergessen
Copy link
Contributor Author

No, if so we should have merged it before the release.. this is a new feature

@PVince81
Copy link
Contributor

PVince81 commented Jul 6, 2015

Fine by me, makes sense.

@jnfrmarks
Copy link

@nickvergessen

Is there any doc for the new apis?

@nickvergessen
Copy link
Contributor Author

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

@jnfrmarks
Copy link

Are there new apis to support the remote shares or is this change an extension of the existing api?

@PVince81
Copy link
Contributor

PVince81 commented Jul 6, 2015

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 ?

@nickvergessen
Copy link
Contributor Author

@jnfrmarks this just adds a non-UI way for remote shares, so we can use them in smashbox to test federated cloud sharing

@nickvergessen
Copy link
Contributor Author

Documentation PR at owncloud-archive/documentation#1329

Feel free to review

@lock lock bot locked as resolved and limited conversation to collaborators Aug 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Share API calls to automatically mount remote shares

8 participants