Skip to content
This repository was archived by the owner on Nov 2, 2018. It is now read-only.

Allow relationships as array#15

Open
qrosmeli wants to merge 6 commits into
stevenpetryk:masterfrom
Beetrack:master
Open

Allow relationships as array#15
qrosmeli wants to merge 6 commits into
stevenpetryk:masterfrom
Beetrack:master

Conversation

@qrosmeli

Copy link
Copy Markdown

Hello.

First, I want to thank you for this code, it save us some time in our migration to JSON API.

I added the option to leave relationships as an array (just like normalizr), with a flag as a second argument (default to false).

@sl-ffx

sl-ffx commented Oct 9, 2018

Copy link
Copy Markdown

@qrosmeli I'm just missing one thing in your PR:

Quote from Readme

Note that, unlike normalizr, this library will always point to associations using the format { type: 'users', id: 4 }.

I think this should be an array of objects with type and id as attributes.

And hopefully @stevenpetryk can merge this PR asap.
This feature would be great to have.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants