ember-data-save-relationships
ember-data-save-relationships copied to clipboard
Reason for disabling embedded relationships
Hi Frank,
Was wondering if there was a reason for disabling embedded relationships?
https://github.com/frank06/ember-data-save-relationships/blob/master/addon/index.js#L42,L43
// do not allow embedded relationships
delete serialized.data.relationships;
Some of my relationships needs that data in order to be saved on the server side. I tried to remove that line and the add-on seems to be working fine. Was wondering if I could safely do this?
Reason: I was lazy to add the test. Does it actually work? Do you think you could contribute a test? If not at some point I'll try myself
Thanks Frank. Will try to add a test for it.
We've got a PR for this - testing on our code base then we will submit. Adds tests and fixes code.
@BryanCrotaz Thanks! Sorry I didn't have the chance to get around to this.
PR #8