Add support to non-jsonapi serializers #7
Conversation
|
@josemarluedke I'm getting an error that |
|
Ahhh, that's true. I got this from our app witch has lodash. I will update this PR to not require it. |
|
@josemarluedke was able to fix with: |
josemarluedke
force-pushed
the
not-only-jsonapi
branch
from
3e1e944 to
f8ef125
Compare
|
@rondale-sc updated removing the usage of lodash. This should be ready for a review/testing/merging. |
|
This PR got a little stale, here's an updated version: https://github.com/CvX/ember-data-fastboot/tree/not-only-jsonapi-updated (a rebase of @josemarluedke's work). |
|
Patiently waiting for this to become ready. |
|
Are the people not using JSONAPI using this commit? Is this going to be merged in at anytime? What are you guys using currently for ember fastboot data? Cheers |
|
@adentranter We are using https://github.com/Appchance/ember-cached-shoe |
|
@medokin legend. Going to look into that tonight :) |
This PR makes possible to serialize and push model data using its serializer, allowing us to not require JSONAPI serializer.
I'm sure the code could be refactored to make things more clear. Please let me know what should I change to make this land.
Also, I did not wanted to follow the idea from #4 of having a custom serializer because ember data is deprecating
store.serializeand I wasn't sure if that deprecation would apply to the approach of #4.This closes #3.