Skip to content

fixing none error #90

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

Closed
wants to merge 2 commits into from
Closed

fixing none error #90

wants to merge 2 commits into from

Conversation

jacoor
Copy link

@jacoor jacoor commented Apr 28, 2015

There is an issue that intercom sometimes spits NONE in response. Flatstorage is dict instance, so will survive none.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 92.06% when pulling 9275638 on ArabellaTech:master into 949056d on jkeyes:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 92.06% when pulling fbe093d on ArabellaTech:master into 949056d on jkeyes:master.

@jkeyes
Copy link
Contributor

jkeyes commented May 9, 2015

@jacoor thanks for the PR. I followed the same rules as the official Ruby wrapper. I'll ask them why they preclude None/nil. I don't see a reason not to.

@jkeyes
Copy link
Contributor

jkeyes commented May 12, 2015

Fixed and now in version 2.0, available from PyPi. See 8ca7d74 for the change and tests.

Thanks again, I've added your profile to the AUTHORS page.

@jkeyes jkeyes closed this May 12, 2015
@jacoor
Copy link
Author

jacoor commented May 12, 2015

Thank You!

2015-05-12 16:22 GMT+02:00 John Keyes notifications@github.com:

Closed #90 #90.

Reply to this email directly or view it on GitHub
#90 (comment).

Jacek Ostański
Internet Evolution - Strony i serwisy Internetowe
608-230-735

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.

3 participants