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