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.

Sign up for GitHub

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

Jump to bottom

fix: JSONAPISerializer.shouldSerializeHasMany relation param type #8819

Merged
merged 1 commit into from
Sep 3, 2023
Merged

fix: JSONAPISerializer.shouldSerializeHasMany relation param type #8819

merged 1 commit into from
Sep 3, 2023

Conversation

samridhivig
Copy link
Contributor

## Description

Reported here: #7996

JSON API Serializer shouldSerializeHasMany third param relation is of type RelationshipSchema.

Notes for the release

@runspired runspired added 🎯 canary PR is targeting canary (default) 🏷️ chore This PR primarily refactors code or updates dependencies 🏷️ doc This PR adds/improves/or fixes documentation and removed 🏷️ chore This PR primarily refactors code or updates dependencies labels Sep 3, 2023
@runspired runspired merged commit a95693d into emberjs:main Sep 3, 2023
32 of 35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@runspired runspired runspired approved these changes

Assignees
No one assigned
Labels
🎯 canary PR is targeting canary (default) 🏷️ doc This PR adds/improves/or fixes documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@samridhivig @runspired