Skip to content

More compatibility fixes#6

Open
chris-hamper wants to merge 9 commits intoboztek:masterfrom
chris-hamper:more_compatibility_fixes
Open

More compatibility fixes#6
chris-hamper wants to merge 9 commits intoboztek:masterfrom
chris-hamper:more_compatibility_fixes

Conversation

@chris-hamper
Copy link
Copy Markdown
Contributor

This includes various fixes and workarounds I've come up with while integrating apps with the JSON API module:

  • Fix findMany() to work with JSON API beta2
  • Prevent dash-ification of relationship keys (JSON API types)
  • (Workaround) Skip serialization of hasMany relationships with no members
  • Fix missing newline

@chris-hamper
Copy link
Copy Markdown
Contributor Author

Again, commits look a bit weird :-P

@chris-hamper
Copy link
Copy Markdown
Contributor Author

I reverted one of the workarounds in the PR, as it probably makes more sense to just implement it in my app until it's fixed in the JSON API module. I'll be working on a patch for that.

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.

1 participant