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

Use type as key for linked resources #728

Merged
merged 1 commit into from
Nov 13, 2014
Merged

Conversation

kurko
Copy link
Member

@kurko kurko commented Nov 12, 2014

If type is author but the association is called writer, the linked
resource key should be called authors, e.g

{
  ...
  linked: {
    authors: [{
      ...
    }]
  }
  ...
}

If type is `author` but the association is called `writer`, the linked
resource key should be called `authors`, e.g

    {
      ...
      linked: {
        authors: [{
          ...
        }]
      }
      ...
    }
@kurko kurko force-pushed the linked-resources-use-type-as-key branch from 3f6a889 to 7333f20 Compare November 12, 2014 22:03
@guilleiguaran
Copy link
Member

LGTM

@sogamoso
Copy link

👍

1 similar comment
@locks
Copy link

locks commented Nov 13, 2014

👍

kurko added a commit that referenced this pull request Nov 13, 2014
@kurko kurko merged commit ac37570 into master Nov 13, 2014
@kurko kurko deleted the linked-resources-use-type-as-key branch November 13, 2014 12:54
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