jsonapi-serializer
jsonapi-serializer copied to clipboard
continuing to clean up dependencies, inflected
Thoughts on continuing to clean up dependencies to reduce bundle size and requirements?
- remove
inflecteddependency forkeyForAttributeand instead default to(k) => k? - remove
inflecteddependency forpluralizeTypeand instead default to(k) => k + 's'?- Users could require
inflectedthemselves and putInflector.dasherizeforkeyForAttributeandInflector.pluralizeforpluralizeType
- Users could require
Continued discussion from #123
Hey @hartzis I think it's a good idea to reduce the bundle size and dependencies. If you have some time, it would be awesome to have a PR. Not sure to have in mind how to implement the "Users could require inflected themselves" :)
@SeyZ since you've already setup config options for keyForAttribute and pluralizeType we could let users just pass Inflector methods to those config options for those, essentially.
I can work on a PR for sure, most of the work would be probably updating tests :)