Documentation update & Broken for relationships about kitsu HOT 3 CLOSED

As for example 2, it seems you encounter an issue when trying to delete a single link in a to-many relationship?
If i'm to interpret the specification, it seems like to-many relationships must supply an array when issuing a DELETE request:

https://jsonapi.org/format/1.2/#crud-updating-to-many-relationships

the client makes a DELETE request to a URL from a relationship link the server MUST delete the specified members from the relationship

Going by the wording, i would say its reasonable to expect the users of Kitsu to supply the argument for api.delete as an array, rather than implementing logic for checking the supplied URL and performing modifications to the body.

I'm happy to PR the changes to the documentation to make this clearer, or add the logic to the delete implementation, but i think @wopian should have final say in whether its within kitsu's scope to include this logic.

You're correct here, which was likely an oversight from me when initialy writing the documentation - the PATCH documentation has its respective equivalents already.

The documentation for api.delete(resource, [123]) should really have had two documentation entries instead of just:

Remove multiple resources (API must support the Bulk Extension)

If you would like to add a PR that adds the following example, that would be great 👍

Remove relationship(s) from a resource
api.delete('posts/relationships/authors', [ 1, 2 ])

I would perhaps put it above the Bulk Extension delete example as that appears to still be an extension-only behaviour in JSON:API 1.1 too.

from kitsu.

Your first example looks to be a simple mistake in the docs, which #782 will fix

As for example 2, it seems you encounter an issue when trying to delete a single link in a to-many relationship?
If i'm to interpret the specification, it seems like to-many relationships must supply an array when issuing a DELETE request:

https://jsonapi.org/format/1.2/#crud-updating-to-many-relationships

If the client makes a DELETE request to a URL from a relationship link the server MUST delete the specified members from the relationship

Going by the wording, i would say its reasonable to expect the users of Kitsu to supply the argument for api.delete as an array, rather than implementing logic for checking the supplied URL and performing modifications to the body.

I'm happy to PR the changes to the documentation to make this clearer, or add the logic to the delete implementation, but i think @wopian should have final say in whether its within kitsu's scope to include this logic.

from kitsu.

Documentation updated in 10.0.3 (2022-10-30)

from kitsu.