API usage - getting diffs

2008-10-06 Thread yop
I think that the title sums it up. I force a 404 error on the demo review-board to get the list if the available API requests but I just can't seem to short out how to get the diff for a specific review. http://code.google.com/p/reviewboard/wiki/ReviewBoardAPI isn't that helpful ;-) Regards,

Re: FAQ additions and (probably) dumb questions

2008-10-06 Thread Christian Hammond
Hi Chris. Answers below. On Mon, Oct 6, 2008 at 5:12 PM, Chris [EMAIL PROTECTED] wrote: I've been playing with RB the last week or so and I'm really impressed. The syntax highlight for diffs is awesome! Glad you like it! I've added (as a comment to the end) a couple of additions to

Re: API usage - getting diffs

2008-10-06 Thread Christian Hammond
On Mon, Oct 6, 2008 at 5:47 PM, Chris Clark [EMAIL PROTECTED] wrote: On Oct 6, 5:26 pm, Christian Hammond [EMAIL PROTECTED] wrote: It's possible in that one could modify the code to add this functionality. However, I wouldn't add support to show the entire raw diff in the upstream

Re: Adding to post-review command line arguments

2008-10-06 Thread Christian Hammond
Best way is to submit a patch so it's used for something in post-review. What flag is this for? Christian -- Christian Hammond - [EMAIL PROTECTED] VMware, Inc. On Mon, Oct 6, 2008 at 6:21 PM, Chris Clark [EMAIL PROTECTED] wrote: post-review has a really elegant mechanism for adding command

Re: Can anybody do review?

2008-10-06 Thread Christian Hammond
It's as designed. The reviewers list is the group of people you're requesting review requests from, but anyone can review. Christian -- Christian Hammond - [EMAIL PROTECTED] VMware, Inc. On Mon, Oct 6, 2008 at 9:53 PM, Atul Phalke [EMAIL PROTECTED]wrote: Hi Team, I created a user John

Issue 624 in reviewboard: Can't add inline comments from diff view discussion tab

2008-10-06 Thread codesite-noreply
Issue 624: Can't add inline comments from diff view discussion tab http://code.google.com/p/reviewboard/issues/detail?id=624 Comment #2 by chipx86: My UI rewrite makes it a little more clear how to respond to a comment, which should in a way satisfy this bug. Essentially, existing comments

Issue 495 in reviewboard: I want a To Me column in the Incoming reviews page

2008-10-06 Thread codesite-noreply
Issue 495: I want a To Me column in the Incoming reviews page http://code.google.com/p/reviewboard/issues/detail?id=495 Comment #4 by trowbrds: (No comment was entered for this change.) Issue attribute updates: Status: Confirmed -- You received this message because you are listed in

Issue 602 in reviewboard: Show the 'basedir' on the review request page

2008-10-06 Thread codesite-noreply
Issue 602: Show the 'basedir' on the review request page http://code.google.com/p/reviewboard/issues/detail?id=602 Comment #2 by trowbrds: I don't really think it's a useful piece of information. All it tells you is what directory someone was in when they ran `svn diff` (or equivalent). This

Issue 682 in reviewboard: Cannot login after fresh install under apache http.

2008-10-06 Thread codesite-noreply
Issue 682: Cannot login after fresh install under apache http. http://code.google.com/p/reviewboard/issues/detail?id=682 New issue report by [EMAIL PROTECTED]: I tried the latest svn version and followed the wiki Getting Started steps to install all necessary stuff. I setup reviewboard to work

Issue 297 in reviewboard: Show selected diff chunk in a better way

2008-10-06 Thread codesite-noreply
Issue 297: Show selected diff chunk in a better way http://code.google.com/p/reviewboard/issues/detail?id=297 Comment #5 by chipx86: Got this working in my UI rewrite. Issue attribute updates: Status: Started Owner: chipx86 -- You received this message because you are listed

Issue 401 in reviewboard: Comments should be inserted into diff below lines being discussed, instead of in dialog box

2008-10-06 Thread codesite-noreply
Issue 401: Comments should be inserted into diff below lines being discussed, instead of in dialog box http://code.google.com/p/reviewboard/issues/detail?id=401 Comment #3 by chipx86: Making the UIs configurable makes it a pain to maintain, document and support. After careful consideration,

Issue 555 in reviewboard: multiple-line selection in a diff doesn't work when selecting upwards

2008-10-06 Thread codesite-noreply
Issue 555: multiple-line selection in a diff doesn't work when selecting upwards http://code.google.com/p/reviewboard/issues/detail?id=555 Comment #3 by chipx86: (No comment was entered for this change.) Issue attribute updates: Labels: Milestone-Release1.0 Usability