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

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

cc @mdibaiee

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

Thanks! 👍

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

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 Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
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
No description provided.
Delete Branch "use-json"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?