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

Update utils.js #1

Merged
merged 1 commit into from
Dec 19, 2016
Merged

Update utils.js #1

merged 1 commit into from
Dec 19, 2016

Conversation

mithoog
Copy link
Contributor

@mithoog mithoog commented Dec 17, 2016

BUG FIXED LINE 35: response is undefined some times, so added check if response object is available

BUG FIXED LINE 35: response is undefined some times, so added check if response object is available
@codecov-io
Copy link

codecov-io commented Dec 17, 2016

Current coverage is 53.01% (diff: 0.00%)

No coverage report found for master at 7205490.

Powered by Codecov. Last update 7205490...1a14a7b

@klarstil
Copy link
Owner

Hey @mithoog, thanks for your PR. Your change looks good to me.

@klarstil klarstil merged commit a8f4b19 into klarstil:master Dec 19, 2016
@klarstil klarstil self-assigned this Dec 19, 2016
@mithoog
Copy link
Contributor Author

mithoog commented Dec 19, 2016

Thank you @klarstil. Deleting my branch.

@mithoog mithoog deleted the patch-1 branch December 19, 2016 08:56
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.

None yet

3 participants