-
Notifications
You must be signed in to change notification settings - Fork 9
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
Improve to-many relations #17
Conversation
Just a little bug and then we should be good. The https://github.com/lode/jsonapi/blob/master/src/response.php#L173 //unset($resource_array['relationships'], $resource_array['meta']);
$this->included_resources = array_merge($this->included_resources, $resource->included_resources); if we comment the And have an Output JSON Like this https://gist.github.com/Pierozi/6361d7f80d6379df472b |
I'm sorry it took so long, but fyi, I'm going to merge and release this PR. I added a separate issue for the bug you noted, as it is not a change from this PR. I plan to pick it up directly though. |
This implements #3 and fixes some issues raised there.