thereadme/hapi-sequelize-crud
1
0
Fork 0
Code Issues 4 Pull Requests 1 Projects Releases Wiki Activity

Fix toJSON responses #24

Merged
joeybaker merged 1 commits from use-json into master 2016-09-08 18:33:48 +00:00
Conversation 3 Commits 1 Files Changed 2
+29 -15
joeybaker commented 2016-09-08 04:06:46 +00:00 (Migrated from github.com)
Copy Link
Copy Source

This is a non-obvious one. Hapi is happy to convert raw sequelize
instances to proper JSON (likely because Sequelize does nice things),
but we do that, we can't use config.response.schema, because it
receives the full sequelize instance instead of JSON.

This is a patch release.

This is a non-obvious one. Hapi is happy to convert raw sequelize instances to proper JSON (likely because Sequelize does nice things), but we do that, we can't use `config.response.schema`, because it receives the full sequelize instance instead of JSON. This is a patch release.
joeybaker commented 2016-09-08 04:07:08 +00:00 (Migrated from github.com)
Copy Link
Copy Source

cc @mdibaiee

cc @mdibaiee
mdibaiee commented 2016-09-08 14:19:38 +00:00 (Migrated from github.com)
Copy Link
Copy Source

Thanks! 👍

Thanks! :+1:
joeybaker commented 2016-09-08 20:28:40 +00:00 (Migrated from github.com)
Copy Link
Copy Source

Published as part of 2.6.2 (tag created for 2.6.1 which is just this change)

Published as part of 2.6.2 (tag created for 2.6.1 which is just this change)
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
thereadme
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: thereadme/hapi-sequelize-crud#24