Skip to content

Conversation

@ninjapenguin
Copy link
Contributor

Spec:

"attributes": an "attributes object", providing information about a resource.

See: http://jsonapi.org/format/#document-structure-resource-objects

Spec: "attributes": an "attributes object", providing information about
a resource.

See: http://jsonapi.org/format/#document-structure-resource-objects
@pmccarren
Copy link
Contributor

@ninjapenguin Has json-api/json-api#588 been released yet? My understanding is that master is headed towards rc4, however it's not quite there yet (I could for sure be wrong though :) )
ref: manyminds/api2go#118

@mjanda
Copy link

mjanda commented Jun 24, 2015

Current ember-data (1.13.4) expects this format, so it should be at least optional and mentioned in readme?

@egeriis
Copy link
Owner

egeriis commented Aug 3, 2015

@ninjapenguin Can you create a new pull request which doesn't conflict with master? :) Thanks.

@egeriis
Copy link
Owner

egeriis commented Aug 19, 2015

Now that JSON API v1 has officially been released, it would be great to have this PR merged. @ninjapenguin do you have the time to fix the merge conflict, please?

@egeriis egeriis mentioned this pull request Sep 21, 2015
@egeriis
Copy link
Owner

egeriis commented Sep 21, 2015

@ninjapenguin Desperately pinging you again :)

@ninjapenguin
Copy link
Contributor Author

@egeriis I'll take a look at this and update the PR this weekend!

@egeriis
Copy link
Owner

egeriis commented Sep 22, 2015

That's just awesome @ninjapenguin 👍

@ninjapenguin
Copy link
Contributor Author

@egeriis This now looks fine to me!

egeriis added a commit that referenced this pull request Sep 28, 2015
updates resource output to correctly nest attributes
@egeriis egeriis merged commit 4e5c29c into egeriis:master Sep 28, 2015
@egeriis
Copy link
Owner

egeriis commented Sep 28, 2015

Thank you so much @ninjapenguin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants