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,
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
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
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
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: 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: 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: 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: 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: 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: 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: 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
12 matches
Mail list logo