search

beauby / jsonapi-datastore

JavaScript client-side JSON API data handling made easy.
MIT License
195 stars 40 forks source link

Add `.findAll(type)` the store class #2

Closed jprincipe closed 8 years ago

beauby commented 8 years ago

Thanks for the PR! I'm currently afk, but will review it later tonight.

beauby commented 8 years ago

LGTM except for the find/findAll mix up in the docs!

jprincipe commented 8 years ago

@beauby updated docementation

jprincipe commented 8 years ago

@beauby I created another branch to convert the library to ES6. I am using babel to generate the dist files so they are converted to ES5, but should make the source code cleaner. Any interest?

https://github.com/jprincipe/jsonapi-datastore/tree/es6

beauby commented 8 years ago

Looks good! Could you squash the commits before I merge? Yeah I'm definitely interested in an ES6 rewrite :+1: