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.

Sign up for GitHub

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

Jump to bottom

Only try to fetch when json_hash is not nil #221

Merged
merged 1 commit into from
Nov 4, 2015
Merged

Only try to fetch when json_hash is not nil #221

merged 1 commit into from
Nov 4, 2015

Conversation

ausmith
Copy link
Contributor

@ausmith ausmith commented Nov 4, 2015

No description provided.

NicholasBusby
NicholasBusby reviewed Nov 4, 2015
View reviewed changes
spec/cb/clients/recommendation_spec.rb
end

include_context :for_company
end

context '.for_company without results' do
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yay test

NicholasBusby added a commit that referenced this pull request Nov 4, 2015
@NicholasBusby
Merge pull request #221 from careerbuilder/catch-nil
99ad061 
Only try to fetch when json_hash is not nil
@NicholasBusby NicholasBusby merged commit 99ad061 into master Nov 4, 2015
@NicholasBusby NicholasBusby deleted the catch-nil branch November 4, 2015 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@ausmith @NicholasBusby