saule icon indicating copy to clipboard operation
saule copied to clipboard

Relationship paths in include not supported

Open PhyberApex opened this issue 7 years ago • 3 comments

As stated https://jsonapi.org/format/#fetching-includes in the JSON API specification. A nested releationship should be included and the dot should be used as a path seperator for such includes.

Currently Saule does not implement this and does not respond with the needed 400 Bad Request message if such an path is encountered.

Same as the other issues I opened. If I find the spare time I would like to open a PR for that.

~Cheers

PhyberApex avatar Oct 16 '18 09:10 PhyberApex

I just took a closer look at this and it seems that nested relationships are not handled at all. I will probably need some assistance to get this to work with the serializer but I will take a close look at this somewhat soonish.

~Cheers

PhyberApex avatar Oct 24 '18 19:10 PhyberApex

Geez with the current implementation this does not seem possible at all. A relationship of a relationship is just not known to Saule...seems like this is impossible to fix...

~Cheers

PhyberApex avatar Oct 19 '19 16:10 PhyberApex

I think this should be possible even if the current implementation does not support it. I have to admit I don't remember all the details of the serializer exactly, but it seems like something that should be supported. I will try to look into it sometime this week.

joukevandermaas avatar Oct 20 '19 15:10 joukevandermaas