Skip to content

Conversation

@denisivan0v
Copy link

@denisivan0v denisivan0v commented Jan 13, 2017

This change is Reviewable

@denisivan0v
Copy link
Author

Related to #66

@commonsensesoftware
Copy link
Collaborator

I didn't forget about your PR. I'm having trouble with setting up the CLA service (which is required). Boo - rules. Shame on me for not getting that setup months ago.

I'm working on a few other related updates. Since the change appears to only require a single method call change, I'll likely just incorporate this modification manually. I do appreciate the help though. Spasiba!

@denisivan0v
Copy link
Author

You're welcome, always glad to help! I'll wait for updates to use them in my apps

@commonsensesoftware
Copy link
Collaborator

The code was right, but there were a few other references that needed to be updated. I'm closing this PR since the actual change will occur in PR #92. You can see the exact changes in commit 448f02e.

I also have the CLA requirements properly set up now, which will enable me to easily accept future contributions from the community. Thanks again for your help.

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