Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/jsonapi relationships #66

Merged
merged 12 commits into from
Jun 26, 2016
Merged

Feature/jsonapi relationships #66

merged 12 commits into from
Jun 26, 2016

Conversation

MP0w
Copy link
Member

@MP0w MP0w commented Jun 25, 2016

Part of #55

@MP0w MP0w added this to the Beta-1 milestone Jun 25, 2016
@codecov-io
Copy link

codecov-io commented Jun 25, 2016

Current coverage is 98.10%

Merging #66 into master will decrease coverage by 0.63%

@@             master        #66   diff @@
==========================================
  Files             6          6          
  Lines           318        369    +51   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits            314        362    +48   
- Misses            4          7     +3   
  Partials          0          0          

Sunburst

Powered by Codecov. Last updated by 0427396...f2be40e

@@ -19,6 +19,15 @@ public protocol JSONAPISerializable {
- returns: Return an object representing the `data` field conforming to JSON API, for `JSONAPIEntity` boxes the return type will be used to fill the `relationships` field otherwise, when nil, the box will be serialized normally and used for the `attributes` field.
*/
func data(includeRelationships includeRelationships: Bool, includeAttributes: Bool) -> AnyObject?

/**
Creates the `included` field by aggregating and unifying the attrbiutes of the relationships recursively
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

attributes

@joanromano
Copy link
Member

👍

@MP0w MP0w merged commit ff84a87 into master Jun 26, 2016
@MP0w MP0w deleted the feature/JSONAPIRelationships branch June 26, 2016 13:53
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.

3 participants