JSONAPISource ajax (findQuery) result errors in success handler about orbit

[dgeb]

@pixelhandler So, as the author of the recent JSON API rewrite, this is rather embarrassing for me...

The fact is that the JSONAPISource is not yet compliant with JSON API. I originally wrote this source as a simple RESTSource, with no concept of primary and linked resources, and with the intention of developing a JSONAPISource separately. However, I grew uncomfortable with the implications of writing such a generic source without a spec to follow, and didn't want to support such a source indefinitely.

I'm planning to get back to this in the next few days, after straightening out some module issues, and before I write my JSON API example project. I'll take a peek at your blog too and might use it as a sanity check for the updated source.

[tute]

I'll be developing my app against memory and localStorage sources meanwhile, knowing that plugging in new sources shouldn't be that hard following stds and predefined APIs. @dgeb if you need any help making JSONAPISource more compliant I'm happy to help, will just need a little guidance at the beginning. Thanks for your work!

[tute]

Question @dgeb: ember-data is highly compliant with JSON API, right? Meaning, if I've got an API working well with ember-data 1.0.0-beta.8 (latest), it is expected that a compliant JSON API Source would work well with it (and will significantly differ from current orbit's implementation)?

As always, thank you so much for your good work.

[dgeb]

@tute I believe this is currently the most JSONAPI-compliant adapter for ember-data: https://github.com/daliwali/ember-json-api

It was recently updated by @stefanpenner to include tests and be more compliant.

[dgeb]

@pixelhandler the underlying incompatibility with JSON API has been fixed in master.

I created pixelhandler/blog#3 to resolve the problem in your branch. You'll need to rebuild orbit and ember-orbit in order to get this working.

Note that JSON API now expects requests for singular resources (such as /posts/:slug) to return singular objects not in arrays. This was a breaking change in the 1.0rc1 rewrite of JSON API. I left this broken, since I thought it would be better to fix your server response rather than customize the orbit serializer.

[pixelhandler]

Awesome @dgeb looking forward to trying it out. Yeah I'll need to change the api as you mentioned. I'm on vacation this week so I'll give it a shot next week. Thanks for the PR!