[PATCH] Add comments REST API
dja at axtens.net
Wed Apr 11 00:56:59 AEST 2018
Stephen Finucane <stephen at that.guru> writes:
> On Mon, 2018-04-09 at 09:29 -0400, Veronika Kabatova wrote:
>> ----- Original Message -----
>> > From: "Stephen Finucane" <stephen at that.guru>
>> > To: vkabatov at redhat.com, patchwork at lists.ozlabs.org
>> > Sent: Monday, April 9, 2018 3:02:57 PM
>> > Subject: Re: [PATCH] Add comments REST API
>> > On Fri, 2018-03-23 at 13:33 +0100, vkabatov at redhat.com wrote:
>> > > From: Veronika Kabatova <vkabatov at redhat.com>
>> > >
>> > > Signed-off-by: Veronika Kabatova <vkabatov at redhat.com>
>> Thanks for feedback!
>> > Looks pretty good. As you note, it does need some tests. I'm also
>> > curious about the idea of nesting this under the patches endpoint. Let
>> > me know what you think.
>> Hmm, that would require the same change for cover letters too. How do
>> you imagine it? Having all related comments under the detailed patch or
>> cover, or something else?
> Indeed. I'd expect all replies to a given patch to be under that patch
> and so on. This is what we present in the web UI so it probably makes
> sense to do the same for the REST API. Unless there's a strong reason
> _not_ to, this is the way I'd go, yeah.
>> I opted for separate API because some people may not care about comments
>> (since AFAIK noone requested it yet), but we very much do. At the same
>> time I didn't want to push our needs on others. If you don't think it's
>> an issue I can rework this and put it under the patch/cover endpoints.
We do have 1 pending request re: the comments API:
I don't know how that would interact with either design, but I wanted to
flag it as a user story to consider.
> Yup, I didn't do it because I wasn't sure how helpful it was. I'm a-ok
> adding it though so go for it.
> PS: You should probably put a 'comments' url in the 'patches' and
> 'covers' resources, similar to what we do for 'checks' of 'patches'.
> Patchwork mailing list
> Patchwork at lists.ozlabs.org
More information about the Patchwork