Issue 605 in reviewboard: current transaction is aborted, commands ignored until end of transaction block

Issue 605: current transaction is aborted, commands ignored until end of transaction block http://code.google.com/p/reviewboard/issues/detail?id=605 Comment #9 by trowbrds: If it works for you, works for me. Issue attribute updates: Status: Fixed -- You received this message because

Issue 586 in reviewboard: Publishing draft of comment shows error.

Issue 586: Publishing draft of comment shows error. http://code.google.com/p/reviewboard/issues/detail?id=586 Comment #1 by trowbrds: It's a bit of a pain, but could you install firebug and get a traceback for this error? At the moment there isn't enough information to know what went wrong. Th

Issue 657 in reviewboard: Show Review Id Number

Issue 657: Show Review Id Number http://code.google.com/p/reviewboard/issues/detail?id=657 Comment #2 by chipx86: Seems a reasonable request. If someone wants to submit a patch for this, we can get it into 1.0. It's actually really easy to add new columns to the dashboard (look in reviews/dat

Issue 658 in reviewboard: Error : The patch to '' didn't apply cleanly.

Issue 658: Error : The patch to '' didn't apply cleanly. http://code.google.com/p/reviewboard/issues/detail?id=658 New issue report by sudarsan: *NOTE: Do not post confidential information in this bug report.* What's the URL of the page containing the problem? http://reviewboard.eng.vmware.com/

Issue 657 in reviewboard: Show Review Id Number

Issue 657: Show Review Id Number http://code.google.com/p/reviewboard/issues/detail?id=657 Comment #1 by olivier.dagenais: Agreed, with the additional reason that traceability can be enhanced when we can more easily refer to reviews by their ID. (i.e. in commit messages, bug reports, status

Issue 657 in reviewboard: Show Review Id Number

Issue 657: Show Review Id Number http://code.google.com/p/reviewboard/issues/detail?id=657 New issue report by bentatham: What's the URL of the page this enhancement relates to, if any? /dashboard and /r/### Describe the enhancement and the motivation for it. Show the Review Id (number) as a

Issue 652 in reviewboard: Mails are not sent when user is in the CC

Issue 652: Mails are not sent when user is in the CC http://code.google.com/p/reviewboard/issues/detail?id=652 Comment #3 by dimitar.kamenov: With your last commit(r1509) it looks fixed. So I don't have to attach you e-mail headers as you request. Just to comment that CC was in the e-mail, but

Issue 394 in reviewboard: if a diff includes a file that has been cvs remove'd, reviewboard won't accept it

Issue 394: if a diff includes a file that has been cvs remove'd, reviewboard won't accept it http://code.google.com/p/reviewboard/issues/detail?id=394 Comment #8 by torsten.flatter: Done: http://reviews.review-board.org/r/555/ -- You received this message because you are listed in the owner

Issue 394 in reviewboard: if a diff includes a file that has been cvs remove'd, reviewboard won't accept it

Issue 394: if a diff includes a file that has been cvs remove'd, reviewboard won't accept it http://code.google.com/p/reviewboard/issues/detail?id=394 Comment #7 by chipx86: Can you please provide this as a diff on http://reviews.review-board.org/ ? Issue attribute updates: Labels: Pe

Issue 652 in reviewboard: Mails are not sent when user is in the CC

Issue 652: Mails are not sent when user is in the CC http://code.google.com/p/reviewboard/issues/detail?id=652 Comment #2 by chipx86: Looks like this was due to not including the CC users in the recipient list that goes out to sendmail. It should be fixed in r1509. Please let me know if it co

Issue 652 in reviewboard: Mails are not sent when user is in the CC

Issue 652: Mails are not sent when user is in the CC http://code.google.com/p/reviewboard/issues/detail?id=652 Comment #1 by chipx86: That patch will add to the BCC list. We need to find out why CC is failing. On the e-mails, are you actually seeing the right e-mail addresses on the "Cc" line?

Issue 394 in reviewboard: if a diff includes a file that has been cvs remove'd, reviewboard won't accept it

Issue 394: if a diff includes a file that has been cvs remove'd, reviewboard won't accept it http://code.google.com/p/reviewboard/issues/detail?id=394 Comment #6 by torsten.flatter: Hello alltogether, I just wrote a patch for the file scmtools/cvs.py This patch will fix that issue. Regards,

Issue 656 in reviewboard: Opera: Clicking links to the actual files in the Diff keeps the cursor on top, rather than navigating to the actual file's diff.

Issue 656: Opera: Clicking links to the actual files in the Diff keeps the cursor on top, rather than navigating to the actual file's diff. http://code.google.com/p/reviewboard/issues/detail?id=656 Comment #1 by chipx86: (No comment was entered for this change.) Issue attribute updates:

Issue 655 in reviewboard: File listing should indicate the type of change

Issue 655: File listing should indicate the type of change http://code.google.com/p/reviewboard/issues/detail?id=655 Comment #2 by chipx86: I'd like to include this type of information, though it won't happen until 1.5 at the earliest. Issue attribute updates: Labels: -Type-Defect Typ