[PATCH v2 07/13] REST: Make 'Patch.state' editable
Stephen Fincane
stephen at that.guru
Fri Nov 25 07:02:49 AEDT 2016
On Mon, 2016-11-21 at 14:36 +1100, Daniel Axtens wrote:
> Hi Stephen,
>
> >
> > This is one of the most useful fields to allow editing of via the
> > API.
> > Make it so.
>
> Please could you revise this commit message?
>
> We seem to already have tests that check if you can successfully
> PATCH
> the state of a patch. This patch can't be making that editable if it
> already is editable.
Commit message revised. As said in the reply to Andy's mail, the test
was broken and didn't validate that we could actually change the field
- only that a HTTP 200 response code would be returned. That, and the
test, are both updated now.
> You also change the way you interact with the state of a patch. I
> quite
> like the change you've made, but the commit message should explain it
> (and that it's a breaking change!).
Good point. Done.
Stephen
More information about the Patchwork
mailing list