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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Hapi to v17 #26

Merged
merged 3 commits into from Jun 11, 2018

Conversation

@kinson
Copy link
Contributor

commented Jun 8, 2018

Overview

Updated Relish to support Hapi v17 馃帀

Changes

  • Updated reference to error details from error.data.details -> error.details
  • Retrieved source from err object (err.output.payload.validation.source)
  • Updated failAction method within validate block to match hapi v17 parameters
  • Updated examples to use hapi 17

Related Issues

Co-authored-by: Fisher Bissett <fisher@dialexa.com>
@kinson kinson self-assigned this Jun 8, 2018
@coveralls

This comment has been minimized.

Copy link

commented Jun 8, 2018

Coverage Status

Coverage remained the same at 100.0% when pulling a752f0b on UpdateToHapi17 into ac9d968 on master.

1 similar comment
@coveralls

This comment has been minimized.

Copy link

commented Jun 8, 2018

Coverage Status

Coverage remained the same at 100.0% when pulling a752f0b on UpdateToHapi17 into ac9d968 on master.

@coveralls

This comment has been minimized.

Copy link

commented Jun 8, 2018

Coverage Status

Coverage remained the same at 100.0% when pulling 465a684 on UpdateToHapi17 into ac9d968 on master.

@kinson kinson added needs review and removed in progress labels Jun 8, 2018
@kinson kinson requested a review from galenandrew Jun 9, 2018
remove node 7 as a supported node version

remove yarn.lock

update documentation and readme
@galenandrew galenandrew force-pushed the UpdateToHapi17 branch from 0a81d03 to 465a684 Jun 11, 2018
@galenandrew galenandrew merged commit a21514e into master Jun 11, 2018
3 checks passed
3 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 100.0%
Details
@galenandrew galenandrew deleted the UpdateToHapi17 branch Jun 11, 2018
galenandrew added a commit that referenced this pull request Jun 11, 2018
Co-authored-by: Fisher Bissett <fisher@dialexa.com>
@galenandrew galenandrew referenced this pull request Jun 11, 2018
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鈥檛 perform that action at this time.