Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Add basic error handling. #30

Merged
merged 3 commits into from Apr 26, 2013

Conversation

Projects
None yet
2 participants
Contributor

cmeiklejohn commented Apr 10, 2013

Don't return empty_response on a PUT failure, return an error tuple and throw a request error. @reiddraper

Contributor

cmeiklejohn commented Apr 11, 2013

@kellymclaughlin @reiddraper Tried dragging this to the working/ready-to-pull column in the kanban and the card vanished. Can we make sure that this gets picked up somehow?

Contributor

kellymclaughlin commented Apr 25, 2013

+1 to merge

cmeiklejohn added a commit that referenced this pull request Apr 26, 2013

@cmeiklejohn cmeiklejohn merged commit fcf296e into develop Apr 26, 2013

@ghost ghost assigned kellymclaughlin Apr 26, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment