search

escalant3 / ember-data-tastypie-adapter

An adapter to connect django applications powered by django-tastypie with ember.js apps
MIT License
102 stars 26 forks source link

Fix for #36 #37

Closed steffann closed 10 years ago

steffann commented 10 years ago

SerializeHasMany doesn't handle promises. Getting the IDs from data avoids the promise and also avoids having to fetch the related records before being able to save a reference to them.

escalant3 commented 10 years ago

Thanks!!

escalant3 commented 10 years ago

I had to revert the PR because some tests were failing with 'undefined' is not an object (evaluating 'get(record, 'data.' + relationship.key).map'). I'll try to have a look when I can.

escalant3 commented 10 years ago

I have also repaired the Travis CI script. If you try to recreate the PR you should see the errors in Github if you cannot run them locally.