- Merge request !9618Very basic way to add arity keys to ResourceObjectData result → (Open) created by GaëlG
- Status changed to Needs work
about 1 month ago 9:41pm 5 March 2025 - 🇫🇷France prudloff Lille
The MR adds the arity key to every resource even if not necessary (for example on responses containing a single resource).
It keeps the code simple and more predictable, but it means a lot of tests had to be updated. - 🇫🇷France prudloff Lille
I answered the questions.
If this needs fixing, we should probably fix it in JsonApiResource as well? - 🇺🇸United States smustgrave
All the tests having to be updated to pass makes me think we are introducing a breaking change
- 🇳🇱Netherlands bbrala Netherlands
Kinda depends, adding a meta to add responses does mean we are doing a change. Is it breaking? If your client support jsonapi it shouldn't be, its part of the spec.
So i lean towards not breaking.
- 🇺🇸United States smustgrave
Can we add an example where it’s not passed to make sure both ways are covered?
- 🇳🇱Netherlands bbrala Netherlands
It's late. Is this also for creating entities? If so, then it should also work without, otherwise it is indeed a BC break since you might not do that :x