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

Support for NextCloud is there #222

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Support for NextCloud is there #222

wants to merge 1 commit into from

Conversation

G-Pereira
Copy link

No description provided.

@PVince81
Copy link
Contributor

hmm support cannot be guaranteed without tests, so might be good to add it to https://github.com/owncloud/pyocclient/blob/master/.travis.yml

I think @blizzz worked on something a while ago

@blizzz
Copy link
Contributor

blizzz commented Oct 11, 2018

True, I forked this to my private repo and had a PR where I was playing with that. Basically, what is need is to set up a nextcloud docker and define env variable for the nextcloud versions. Likely the travis conf needs to be tuned a bit to play along with both server software. As a blue print it might be good enough, cf: https://github.com/blizzz/pyocclient/pull/3/files

@G-Pereira would you look into adjusting the travis config?

@G-Pereira
Copy link
Author

I've tested throughly on NextCLoud, it works

@G-Pereira
Copy link
Author

Please do it @blizzz

@landryb
Copy link

landryb commented Jun 7, 2019

I'm using (and abusing!) it against nextcloud 15, and it works fine so far.

@sunjam
Copy link

sunjam commented Oct 16, 2019

For Nextcloud users, should we just use this PR until it is merged, or should we use the PR that @blizzz linked?

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.

5 participants