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

Use hoek to assign defaults to options passed to inject #22

Closed
phated opened this issue Dec 11, 2014 · 2 comments
Closed

Use hoek to assign defaults to options passed to inject #22

phated opened this issue Dec 11, 2014 · 2 comments
Labels
bug
Milestone

Comments

@phated
Copy link
Contributor

@phated phated commented Dec 11, 2014

Currently, if method isn't defined on the options passed in, it assigns a method property as 'GET'. This should be done using hoek (or some simpler form) so it doesn't mutate the incoming options object.

@hueniverse hueniverse added the bug label Dec 11, 2014
arb added a commit to arb/shot that referenced this issue Jan 8, 2015
@bnjmnt4n

This comment has been minimized.

Copy link

@bnjmnt4n bnjmnt4n commented Feb 18, 2015

👍 Any updates?

@hueniverse

This comment has been minimized.

Copy link
Member

@hueniverse hueniverse commented Feb 19, 2015

@chrisdickinson chrisdickinson added this to the 1.4.1 milestone Feb 19, 2015
@arb arb closed this in b314e73 Feb 19, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.