Vesa Laakso

Results 287 comments of Vesa Laakso

Just checking up @jeremyjaybaker, is this issue still timely for you?

Ok JR 0.10 has been released now. It seems that the major changes done are things concerning jsonapi-authorization, so it will take quite an effort to get us compatible with...

Not really, sorry. One thing I'd really like to see here would be a new approach to tests where we would assert on user visible functionality — so going from...

Does not seem like it. Feel free to offer help if you need JR 0.10 support.

Thank you for offering to help, @lgebhardt ☺️. I haven't looked into the compatibility differences much either — I merely know that the way we've used to hook into the...

I discovered this while writing docs in #71 and this inconsistency struck out a bit. I added a [TODO comment in the upcoming docs](https://github.com/venuu/jsonapi-authorization/blob/d364cbc65860e44722046a84aceb708e6ad95266/docs/relationship-authorization.md#removing-a-has-many-relationship) for now.

Hmm I guess that if we change this, we should also change how clearing out the has-many relationship with a PATCH on the resource itself is handled: > `PATCH /articles/article-1`...

@ouabing can you show an example scenario where there's breakage? Preferably open a new issue with that information :relaxed:

Thanks for clarifying :relaxed:. I hid the comments above as resolved as this discussion wasn't about the original issue description

Hi, thanks for the question! Sorry for my response being delayed, as I was on vacation. Returning 404 is actually the intended behavior currently. I agree that it can be...