Skip to content
This repository was archived by the owner on Sep 24, 2018. It is now read-only.

Conversation

@joehoyle
Copy link
Member

This is currently difficult to do, as you have to send one request for each
associtation. Instead, this PR makes it possible to POST to
/wp/v2/posts/123/terms/category with { ids: [1,2,4] }.

We could also add a names param to assign multiple terms via Name, which is
an often requested feature we don't currently support.

This is currently difficult to do, as you have to send one request for
each associtation. Instead, this PR makes it possible to POST to
`/wp/v2/posts/123/terms/category` with `{ ids: [1,2,4] }`.

We could also add a `names` param to assign multiple terms via Name,
which is an often requested feature we don't currently support.
@joehoyle joehoyle added this to the 2.0 Beta 4 milestone Jul 24, 2015
@joehoyle joehoyle self-assigned this Jul 24, 2015
@TimothyBJacobs
Copy link
Member

Why use an append keyword instead of using the differences between the HTTP Verbs PUT and PATCH?

@rmccue
Copy link
Member

rmccue commented Jul 24, 2015

Why use an append keyword instead of using the differences between the HTTP Verbs PUT and PATCH?

Supporting PATCH is a pretty major undertaking, since PATCH isn't just a partial update. We'd need to support something like JSON Patch fully to do it. (I don't think that's a bad idea, but it does mean a fair amount of work.)

See also http://williamdurand.fr/2014/02/14/please-do-not-patch-like-an-idiot/

Re: the main issue, on my list to look at and review. :)

@rmccue rmccue modified the milestones: 2.0, 2.0 Beta 4 Aug 7, 2015
@danielbachhuber danielbachhuber removed this from the 2.0 milestone Jan 21, 2016
@danielbachhuber
Copy link
Member

Problem solved in #2063

@danielbachhuber danielbachhuber deleted the add-multiple-terms-to-post branch January 21, 2016 16:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants