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

Add full response for directions api #10

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jguthrie100
Copy link

Added a feature that allows 'return_type: :all' to be passed into the client.directions method, which returns the full Hash output, rather than just the [:routes] entry

Can change the argument to be 'return_type: :status' if you just want the status, or leave it blank to get the [:routes] value (i.e. the same output as is the case without this change)

 - When doing a directions search, if you enter an invalid address (i.e. 'dsadsfdfdscfrcerf, Tanzania'), a NotFoundError is raised
…amuel#6)

Added a feature that allows 'return_type: :all' to be passed into the client.directions method, which returns the full Hash output, rather than just the [:routes] entry

Can change the argument to be 'return_type: :status' if you just want the status, or leave it blank to get the [:routes] value (i.e. the same output as is the case without this change)
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.3%) to 99.73% when pulling a86563b on jguthrie100:add_full_response_for_directions_api into 46746fd on edwardsamuel:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.3%) to 99.73% when pulling a86563b on jguthrie100:add_full_response_for_directions_api into 46746fd on edwardsamuel:master.

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