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

support many strategies #114

Merged
merged 5 commits into from Feb 22, 2016
Merged

support many strategies #114

merged 5 commits into from Feb 22, 2016

Conversation

@jaw187
Copy link
Contributor

jaw187 commented Feb 19, 2016

Fixes #111

@jaw187 jaw187 added the feature label Feb 19, 2016
@jaw187 jaw187 added this to the 6.1.0 milestone Feb 19, 2016
@jaw187 jaw187 self-assigned this Feb 19, 2016
@jaw187

This comment has been minimized.

Copy link
Contributor Author

jaw187 commented Feb 19, 2016

@hueniverse any concerns with allowing clients to specify the name of the decoration applied to request?

@hueniverse

This comment has been minimized.

Copy link
Member

hueniverse commented Feb 20, 2016

Nope. I would be concerned with keeping track of everything on the app side but that's a different issue.

jaw187 added a commit that referenced this pull request Feb 22, 2016
support many strategies
@jaw187 jaw187 merged commit f3d117a into hapijs:master Feb 22, 2016
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.