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

bug

duplicate

enhancement

help wanted

invalid

question

wontfix

Milestone

Clear milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

No Assignees

1 Participants

Notifications

Subscribe

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#24

No description provided.