search

scoutforpets / jsonapi-mapper

JSON API-Compliant Serialization for your Node ORM
The Unlicense
42 stars 24 forks source link

mapper breaks bookshelf visibility plugin #76

Closed jamesdixon closed 8 years ago

jamesdixon commented 8 years ago

The visibility plugin allows you to hide fields from the response (ex: password). Currently, it doesn't work with the mapper. Looks like it's part of the toJSON method, which we're overriding:

https://github.com/tgriesser/bookshelf/blob/master/src/plugins/visibility.js

jamesdixon commented 8 years ago

@chamini2 what do you think about getting this and #35 resolved for 1.0? Both are core bookshelf plugins.

chamini2 commented 8 years ago

Agreed, we may be in beta for a little while 😄 .