Boom #4

Merged
joeybaker merged 3 commits from boom into master 2016-06-30 13:44:07 +00:00
joeybaker commented 2016-06-30 04:32:44 +00:00 (Migrated from github.com)
  • Adds boom to reply with 404s when necessary
  • fixes a joi requirement that id's must be numbers; primary keys can also be strings
- Adds boom to reply with 404s when necessary - fixes a joi requirement that id's must be numbers; primary keys can also be strings
mdibaiee commented 2016-06-30 13:44:12 +00:00 (Migrated from github.com)

Hey, thanks for the pull-request. Seems fine to me, merging! 👍

Hey, thanks for the pull-request. Seems fine to me, merging! :+1:
joeybaker commented 2016-06-30 13:49:01 +00:00 (Migrated from github.com)

Cool, thanks! We might have some more coming your way - we just started using this and are pretty happy with it. Thanks for building it!

Cool, thanks! We might have some more coming your way - we just started using this and are pretty happy with it. Thanks for building it!
mdibaiee commented 2016-06-30 13:54:37 +00:00 (Migrated from github.com)

@joeybaker: That would be great! Looking forward to your pull-requests. You're welcome, I'm glad you found it useful. 😀

@joeybaker: That would be great! Looking forward to your pull-requests. You're welcome, I'm glad you found it useful. 😀
mdibaiee commented 2016-06-30 13:59:18 +00:00 (Migrated from github.com)

published to npm hapi-sequelize-crud@2.1.0. 07af23cb5a

published to npm `hapi-sequelize-crud@2.1.0`. https://github.com/mdibaiee/hapi-sequelize-crud/commit/07af23cb5a30a7466e42508c602db92311fa932d
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: thereadme/hapi-sequelize-crud#4
No description provided.