Page MenuHomePhabricator

add diff=stable to API
Closed, DeclinedPublic

Description

Please allow the diff=review url also with api. Thanks.


Version: unspecified
Severity: enhancement

Details

Reference
bz24036

Event Timeline

bzimport raised the priority of this task from to Lowest.Nov 21 2014, 11:10 PM
bzimport set Reference to bz24036.
bzimport added a subscriber: Unknown Object (MLST).

This would really fit all with prop=revisions. What api page is this for?

(In reply to comment #1)

This would really fit all with prop=revisions. What api page is this for?

It should be for prop=revisions rvdiffto. But I get <error code="rvdiffto" info="rvdiffto must be set to a non-negative number, "prev", "next" or "cur"" xml:space="preserve">

diff=review is for just the stable to the current. This API parameter is for comparing a list of revisions to rvdiffto. These aren't congruent it all. It would make more sense to have rvdiffto=stable if anything.

Is a rvdiffto=stable param OK?

(In reply to comment #5)

Is a rvdiffto=stable param OK?

Not enough info or support at the moment.

(In reply to comment #5)

Is a rvdiffto=stable param OK?

Yes, it is okay.