James Le Cuirot
James Le Cuirot
Since writing that, I've realised that `Her::Model::ORM#build` creates the request parameters directly from the given attributes hash rather than creating a new resource instance first so this problem is slightly...
Yes, I do use `send_only_modified_attributes` already but I don't think that matters? Modifying the associated data should not control whether it is included in the parent request or not. The...
Separate endpoints in my case but I'm trying to consider the bigger picture here. I want to make Her work better for everyone.
I think I've finally got something workable now. I decided to exclude association data from build requests when the foreign key is present, e.g. if service_id is present then the...
Okay, here it is. This goes quite a but further than the original pull request but the changes are at least loosely related and submitting them separately would have been...
IIRC this will conflict with the second commit of #298. This side should win but I can fix the conflict myself once you merge one of these.
I've made some relatively small tweaks to this but I'll give them a bit more testing before I push up again.
Small tweaks ballooned into bigger changes. With the build requests working as I intended, some parameters used to build the path started appearing in the query string so I had...
Drat, after months of no issues and heavy use in production, my colleague ran into an infinite loop involving `to_json`. It looks very much like the earlier problem with `inspect`....
It's not that. The new code does a better job of caching associated resources so you can easily end up with loops, hence why I had to fix this for...