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

Collaboration refactoring to use CollabWrapper #4

Merged
merged 1 commit into from Feb 18, 2016

Conversation

Projects
None yet
3 participants
@erilyth
Copy link
Contributor

commented Jan 8, 2016

Move to the CollabWrapper offered by @samdroid-apps, https://github.com/samdroid-apps/collabwrapper

@samdroid-apps

This comment has been minimized.

Copy link
Member

commented Jan 9, 2016

Heh github decided to cc me into this. It looks really good. But remember to add the standalone version. I fixed up the guide for how to do that and put it here: https://github.com/samdroid-apps/collabwrapper

@erilyth

This comment has been minimized.

Copy link
Contributor Author

commented Jan 9, 2016

I added 'from collabwrapper.collabwrapper import CollabWrapper' as the alternative import. Is this fine?

@samdroid-apps

This comment has been minimized.

Copy link
Member

commented Jan 9, 2016

Yep, but you need to add the submodule, right now it isn't included in the commit.

@erilyth erilyth force-pushed the erilyth:collabstory branch from 29ee246 to e1a4955 Jan 10, 2016

@erilyth

This comment has been minimized.

Copy link
Contributor Author

commented Jan 10, 2016

@samdroid-apps done :)

walterbender added a commit that referenced this pull request Feb 18, 2016

Merge pull request #4 from erilyth/collabstory
Collaboration refactoring to use CollabWrapper

@walterbender walterbender merged commit e8edd93 into sugarlabs:master Feb 18, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.