Reviewboard Plugin Broken?

Aleix Pol aleixpol at kde.org
Tue Oct 4 22:07:12 UTC 2011


On Tue, Oct 4, 2011 at 11:35 PM, David Narvaez
<david.narvaez at computer.org>wrote:

> On Tue, Oct 4, 2011 at 7:25 AM, Aleix Pol <aleixpol at kde.org> wrote:
> > Maybe they changed the API?
> > I'd be very interested in helping you fix this :)
> > Aleix
>
> It does sound like an API change, but what looks weird to me is that
> the API we were using (which I _know_ was working) is not documented
> anywhere in the Reviewboard documentation.
>
> My first idea was to modify the URL and use the Accept header to
> request a JSON response. That worked, but the default number of
> repositories that are included in the response is 25. You need an
> extra request to get the total count first and then include that count
> in the second request (both, according to the documentation, should be
> GET requests) so that's the workflow I'm planning to implement. If so,
> then I wouldn't be using the HttpPostCall but doing everything inside
> the ProjectRequest. Makes any sense?
>
Well, if they require it, it makes full sense.


>
> Anyhow, I found this bug
>
> https://bugs.kde.org/show_bug.cgi?id=270677
>
> which is technically not the same issue as we have now (because I
> remember it didn't work before, when I was also using KDevelop 4.2, I
> remember it working in KDevelop 4.3 with latest KDevPlatform and now
> it doesn't work again) but I think we could use that space to discuss
> further if needed.
>

It's ok, you can also open a new bug, it's cheap!


>
> David E. Narváez
>
> --
> KDevelop-devel mailing list
> KDevelop-devel at kdevelop.org
> https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel
>


Aleix
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20111005/46e1a2cd/attachment.html>


More information about the KDevelop-devel mailing list