You want this format now: http:// <rbserver>/api/review-requests/?changenum=<changenum>&repository=<repository_id>
That will return a list matching that criteria. The first item will be what you want. You can use that data as-is or follow the links in the payload. Christian -- Christian Hammond - [email protected] Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Mon, Sep 26, 2011 at 11:23 PM, jack jack <[email protected]> wrote: > Hi, > > As per your suggestion i browse the tree at /api and also at > /api/review-requests/ but don't see review id also the changenum. Could you > please provide me complete HTTP URL where should i get the review ID for > changenum. > > I may be missing lot of thing here ....but after upgrading the RB 1.0 to > 1.6.1 my wrapper which was expecting some of the api failing and i need to > write complete wrapper again. > > Please help me or provide me pointer. > > Here is my existing subroutine which > > def get_review_request_by_changenum(self, changenum): > """ > Get the review request object via a perforce changelist > > Args: 'changenum' - perforce changelist > json call: http:// > <reviewboard>/api/json/reviewrequests/repository/1/<changenum>/ > """ > if changenum == 'default': > print "default not allowed. please a pending or submitted > changelist" > sys.exit(1) > self.myurl = self.rbserver \ > + '/api/json/reviewrequests/repository/1/changenum/' > \ > + changenum + '/' > self.urlopen = self.opener.open(self.myurl,self.data) > > self.result = simplejson.load(self.urlopen) > > if self.result['stat'] == "ok": > self.dict = self.result['review_request'] > return self.dict > else: > return > > Please correct me for the above code....I got blocked. > > Regards, > Jack > > On Tue, Sep 27, 2011 at 12:05 AM, Christian Hammond > <[email protected]>wrote: > >> I don't have any sample code to show you. It's just an HTTP GET at the >> URL, though. You can browse the tree at /api/. >> >> post-review may be useful for some bits of that. >> >> >> Christian >> >> -- >> Christian Hammond - [email protected] >> Review Board - http://www.reviewboard.org >> VMware, Inc. - http://www.vmware.com >> >> >> On Mon, Sep 26, 2011 at 11:30 AM, jack jack <[email protected]> wrote: >> >>> Thanks Christian, >>> >>> Could you please point me with sample code for the usage of these new >>> webapi. >>> >>> On Mon, Sep 26, 2011 at 9:53 PM, Christian Hammond >>> <[email protected]>wrote: >>> >>>> Hi Jack, >>>> >>>> No, that old API was removed. This will work, though: >>>> >>>> >>>> http://www.reviewboard.org/docs/manual/1.5/webapi/2.0/resources/review-request-list/ >>>> >>>> Basically, you can pass ?changenum= to get a list with that changenum >>>> (and you can also narrow down by repository and other criteria). >>>> >>>> Christian >>>> >>>> -- >>>> Christian Hammond - [email protected] >>>> Review Board - http://www.reviewboard.org >>>> VMware, Inc. - http://www.vmware.com >>>> >>>> >>>> On Mon, Sep 26, 2011 at 4:16 AM, jack jack <[email protected]> wrote: >>>> >>>>> I am writing a wrapper to get the review-request by changenum. >>>>> >>>>> I wanted to know, does this json api call is still valid for RB server >>>>> 1.6. >>>>> >>>>> >>>>> http://www.reviewboard.org/docs/manual/1.5/webapi/1.0/reference/apicalls/get-review-request-by-changenum/ >>>>> >>>>> if not then please point me to the version where i can get the similar >>>>> operation. >>>>> >>>>> Thanks in advance. >>>>> >>>>> -- >>>>> >>>>> Want to help the Review Board project? Donate today at >>>>> http://www.reviewboard.org/donate/ >>>>> Happy user? Let us know at http://www.reviewboard.org/users/ >>>>> -~----------~----~----~----~------~----~------~--~--- >>>>> To unsubscribe from this group, send email to >>>>> [email protected] >>>>> For more options, visit this group at >>>>> http://groups.google.com/group/reviewboard?hl=en >>>>> >>>> >>>> -- >>>> Want to help the Review Board project? Donate today at >>>> http://www.reviewboard.org/donate/ >>>> Happy user? Let us know at http://www.reviewboard.org/users/ >>>> -~----------~----~----~----~------~----~------~--~--- >>>> To unsubscribe from this group, send email to >>>> [email protected] >>>> For more options, visit this group at >>>> http://groups.google.com/group/reviewboard?hl=en >>>> >>> >>> -- >>> Want to help the Review Board project? Donate today at >>> http://www.reviewboard.org/donate/ >>> Happy user? Let us know at http://www.reviewboard.org/users/ >>> -~----------~----~----~----~------~----~------~--~--- >>> To unsubscribe from this group, send email to >>> [email protected] >>> For more options, visit this group at >>> http://groups.google.com/group/reviewboard?hl=en >>> >> >> -- >> Want to help the Review Board project? Donate today at >> http://www.reviewboard.org/donate/ >> Happy user? Let us know at http://www.reviewboard.org/users/ >> -~----------~----~----~----~------~----~------~--~--- >> To unsubscribe from this group, send email to >> [email protected] >> For more options, visit this group at >> http://groups.google.com/group/reviewboard?hl=en >> > > -- > Want to help the Review Board project? Donate today at > http://www.reviewboard.org/donate/ > Happy user? Let us know at http://www.reviewboard.org/users/ > -~----------~----~----~----~------~----~------~--~--- > To unsubscribe from this group, send email to > [email protected] > For more options, visit this group at > http://groups.google.com/group/reviewboard?hl=en > -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~----------~----~----~----~------~----~------~--~--- To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en
