Issue 2038 in reviewboard: Automatic hyperlink regexes are broken

Status: New Owner: Labels: Type-Defect Priority-Medium New issue 2038 by sbress...@gmail.com: Automatic hyperlink regexes are broken http://code.google.com/p/reviewboard/issues/detail?id=2038 * NOTE: Do not post confidential information in this bug report. * * If you need

Re: Issue 1998 in reviewboard: Error when visiting user profile page: [Django] Error (EXTERNAL IP): /users/

Updates: Status: Fixed Comment #1 on issue 1998 by trowb...@gmail.com: Error when visiting user profile page: [Django] Error (EXTERNAL IP): /users/ http://code.google.com/p/reviewboard/issues/detail?id=1998 Oops, this page was doing something monumentally silly. Fixed in master

Re: Issue 2038 in reviewboard: Automatic hyperlink regexes are broken

Updates: Status: Duplicate Mergedinto: 2002 Comment #1 on issue 2038 by trowb...@gmail.com: Automatic hyperlink regexes are broken http://code.google.com/p/reviewboard/issues/detail?id=2038 (No comment was entered for this change.) -- You received this message because you

Re: Issue 2002 in reviewboard: linkified URL has succeeding parenthesis included in link's target URL

Comment #1 on issue 2002 by trowb...@gmail.com: linkified URL has succeeding parenthesis included in link's target URL http://code.google.com/p/reviewboard/issues/detail?id=2002 Issue 2038 has been merged into this issue. -- You received this message because you are subscribed to the Google

Re: Issue 1648 in reviewboard: post-review throws an exception for Perforce changesets with huge descriptions.

Updates: Status: Duplicate Mergedinto: 1481 Comment #2 on issue 1648 by trowb...@gmail.com: post-review throws an exception for Perforce changesets with huge descriptions. http://code.google.com/p/reviewboard/issues/detail?id=1648 (No comment was entered for this change.) --

Re: Issue 1481 in reviewboard: Review Board 500 error if testing done is over 65535 characters (or a bit smaller)

Comment #4 on issue 1481 by trowb...@gmail.com: Review Board 500 error if testing done is over 65535 characters (or a bit smaller) http://code.google.com/p/reviewboard/issues/detail?id=1481 Issue 1648 has been merged into this issue. -- You received this message because you are subscribed

Re: Issue 2024 in reviewboard: MultipleObjectsReturned: get() returned more than one ReviewRequest

Updates: Status: Fixed Comment #1 on issue 2024 by trowb...@gmail.com: MultipleObjectsReturned: get() returned more than one ReviewRequest http://code.google.com/p/reviewboard/issues/detail?id=2024 Fixed in release-1.5.x. This will ship with 1.5.5. -- You received this message

Re: Issue 1791 in reviewboard: Hide unreasonably large diffs.

Updates: Status: WontFix Comment #2 on issue 1791 by trowb...@gmail.com: Hide unreasonably large diffs. http://code.google.com/p/reviewboard/issues/detail?id=1791 I think this is actually a social problem rather than a technical one. If your users are posting code reviews of

Re: Issue 1695 in reviewboard: filename containing spaces is not handled properly in UI

Updates: Status: Duplicate Mergedinto: 1533 Comment #2 on issue 1695 by trowb...@gmail.com: filename containing spaces is not handled properly in UI http://code.google.com/p/reviewboard/issues/detail?id=1695 (No comment was entered for this change.) -- You received this

Re: Issue 1533 in reviewboard: File Names Cut off in Diff Viewer

Comment #5 on issue 1533 by trowb...@gmail.com: File Names Cut off in Diff Viewer http://code.google.com/p/reviewboard/issues/detail?id=1533 Issue 1695 has been merged into this issue. -- You received this message because you are subscribed to the Google Groups reviewboard-issues group. To

Re: Issue 1633 in reviewboard: Repository configuration should not expose passwords

Updates: Status: WontFix Comment #9 on issue 1633 by trowb...@gmail.com: Repository configuration should not expose passwords http://code.google.com/p/reviewboard/issues/detail?id=1633 After thinking about this for a while, I think the solution is use https -- You received this

Re: Issue 1724 in reviewboard: error when trying to view diff for create_gse_esl.mac

Updates: Status: UserError Comment #1 on issue 1724 by trowb...@gmail.com: error when trying to view diff for create_gse_esl.mac http://code.google.com/p/reviewboard/issues/detail?id=1724 This is something that you'll have to work with your server admin to track down. Typically

Re: Issue 1983 in reviewboard: Include the server's time zone in the Web API

Updates: Status: Fixed Comment #1 on issue 1983 by trowb...@gmail.com: Include the server's time zone in the Web API http://code.google.com/p/reviewboard/issues/detail?id=1983 Fixed in release-1.5.x as 529227b. This will ship in 1.5.5. Thanks! -- You received this message because

Re: Issue 774 in reviewboard: Review tracker for user

Comment #7 on issue 774 by trowb...@gmail.com: Review tracker for user http://code.google.com/p/reviewboard/issues/detail?id=774 I'm marking bug 1798 as a dupe, because it's profoundly similar, but it's asking for the ability to hide files in a diff behind disclosure triangles (which could

Re: Issue 1798 in reviewboard: Allow diffs on view diff page to be folded.

Updates: Status: Duplicate Mergedinto: 774 Comment #2 on issue 1798 by trowb...@gmail.com: Allow diffs on view diff page to be folded. http://code.google.com/p/reviewboard/issues/detail?id=1798 This is similar enough to bug 774 that I'm going to combine them. -- You received

Re: Issue 1747 in reviewboard: post-review --revison-range=XXXXX crashes (implement post-commit posting)

Updates: Status: Duplicate Mergedinto: 921 Comment #3 on issue 1747 by trowb...@gmail.com: post-review --revison-range=X crashes (implement post-commit posting) http://code.google.com/p/reviewboard/issues/detail?id=1747 (No comment was entered for this change.) -- You

Re: Issue 921 in reviewboard: Add post-commit review submission to post-review for perforce

Comment #3 on issue 921 by trowb...@gmail.com: Add post-commit review submission to post-review for perforce http://code.google.com/p/reviewboard/issues/detail?id=921 Issue 1747 has been merged into this issue. -- You received this message because you are subscribed to the Google Groups

Re: Issue 1390 in reviewboard: Review emails should have Review ID # in title/subject of email

Updates: Status: WontFix Comment #3 on issue 1390 by trowb...@gmail.com: Review emails should have Review ID # in title/subject of email http://code.google.com/p/reviewboard/issues/detail?id=1390 I'm going to go ahead and claim that your desires are specific enough that you should

Re: Issue 1513 in reviewboard: Update diff : Error: Saving the form failed due to a server error: 0 n/a

Updates: Status: Fixed Comment #5 on issue 1513 by trowb...@gmail.com: Update diff : Error: Saving the form failed due to a server error: 0 n/a http://code.google.com/p/reviewboard/issues/detail?id=1513 This has changed significantly since 1.0.x, and I haven't seen any reports of

Re: Issue 1433 in reviewboard: DiffViewer Files Changed list is incomplete when png images are on changelist

Updates: Status: Fixed Comment #2 on issue 1433 by trowb...@gmail.com: DiffViewer Files Changed list is incomplete when png images are on changelist http://code.google.com/p/reviewboard/issues/detail?id=1433 I believe this was fixed in another change. -- You received this message

Re: Issue 1183 in reviewboard: Viewing diff in review request causes traceback involving memcache

Updates: Status: Duplicate Mergedinto: 387 Comment #11 on issue 1183 by trowb...@gmail.com: Viewing diff in review request causes traceback involving memcache http://code.google.com/p/reviewboard/issues/detail?id=1183 (No comment was entered for this change.) -- You received

Re: Issue 387 in reviewboard: Error Control Characters not allowed

Comment #7 on issue 387 by trowb...@gmail.com: Error Control Characters not allowed http://code.google.com/p/reviewboard/issues/detail?id=387 Issue 1183 has been merged into this issue. -- You received this message because you are subscribed to the Google Groups reviewboard-issues group.

Re: Issue 387 in reviewboard: Error Control Characters not allowed

Updates: Status: Fixed Comment #8 on issue 387 by trowb...@gmail.com: Error Control Characters not allowed http://code.google.com/p/reviewboard/issues/detail?id=387 Fixed in release-1.5.x as 171d857. This will ship in 1.5.5. Thanks! -- You received this message because you are

Re: Issue 1770 in reviewboard: Provide feedback on post-review progress

Updates: Status: WontFix Comment #2 on issue 1770 by trowb...@gmail.com: Provide feedback on post-review progress http://code.google.com/p/reviewboard/issues/detail?id=1770 There are only really two places where it can block--talking to the SCM, and talking to review board. In

Re: Issue 1263 in reviewboard: Add file navigation popup list to view diffs page

Comment #2 on issue 1263 by trowb...@gmail.com: Add file navigation popup list to view diffs page http://code.google.com/p/reviewboard/issues/detail?id=1263 Issue 1591 has been merged into this issue. -- You received this message because you are subscribed to the Google Groups

Re: Issue 1591 in reviewboard: Improve the navigation by providing a vertical bar showing where you are, and where diffs are

Updates: Status: Duplicate Mergedinto: 1263 Comment #4 on issue 1591 by trowb...@gmail.com: Improve the navigation by providing a vertical bar showing where you are, and where diffs are http://code.google.com/p/reviewboard/issues/detail?id=1591 (No comment was entered for

Re: Issue 1794 in reviewboard: alert (popup) the user when attempting to leave a page with unpublished changes

Comment #2 on issue 1794 by trowb...@gmail.com: alert (popup) the user when attempting to leave a page with unpublished changes http://code.google.com/p/reviewboard/issues/detail?id=1794 Issue 1706 has been merged into this issue. -- You received this message because you are subscribed to

Re: Issue 1706 in reviewboard: View Diff causes un-saved changes to Description to be dropped

Updates: Status: Duplicate Mergedinto: 1794 Comment #2 on issue 1706 by trowb...@gmail.com: View Diff causes un-saved changes to Description to be dropped http://code.google.com/p/reviewboard/issues/detail?id=1706 1794 characterizes it better. -- You received this message

Re: Issue 2014 in reviewboard: get_interesting_headers raises IndexError with C++ end-of-file whitespace change

Updates: Status: Fixed Comment #1 on issue 2014 by trowb...@gmail.com: get_interesting_headers raises IndexError with C++ end-of-file whitespace change http://code.google.com/p/reviewboard/issues/detail?id=2014 Fixed in release-1.5.x as beda363. This will ship in 1.5.5. Thanks! --

Re: Issue 2033 in reviewboard: System is broken after clicking save in File Storage

Updates: Status: Duplicate Mergedinto: 1888 Comment #1 on issue 2033 by trowb...@gmail.com: System is broken after clicking save in File Storage http://code.google.com/p/reviewboard/issues/detail?id=2033 (No comment was entered for this change.) -- You received this message

Re: Issue 1888 in reviewboard: KeyError at /review/admin/db/scmtools/repository/add/

Comment #4 on issue 1888 by trowb...@gmail.com: KeyError at /review/admin/db/scmtools/repository/add/ http://code.google.com/p/reviewboard/issues/detail?id=1888 Issue 2033 has been merged into this issue. -- You received this message because you are subscribed to the Google Groups

Re: Issue 1888 in reviewboard: KeyError at /review/admin/db/scmtools/repository/add/

Updates: Status: Fixed Comment #5 on issue 1888 by trowb...@gmail.com: KeyError at /review/admin/db/scmtools/repository/add/ http://code.google.com/p/reviewboard/issues/detail?id=1888 Fixed in release-1.5.x as 9bdc33f. Thanks! -- You received this message because you are subscribed

Re: Issue 1983 in reviewboard: Include the server's time zone in the Web API

Comment #2 on issue 1983 by robert.munteanu: Include the server's time zone in the Web API http://code.google.com/p/reviewboard/issues/detail?id=1983 Thanks! Are you sure the commit is correct? https://github.com/reviewboard/reviewboard/commit/529227b seems unrelated. -- You received

Re: Issue 2035 in reviewboard: document.title (tab name) should include bug number and submitter

Comment #2 on issue 2035 by keshav.a...@gmail.com: document.title (tab name) should include bug number and submitter http://code.google.com/p/reviewboard/issues/detail?id=2035 If you have 5 tabs open with 5 reviews, and you have to click through all of them just to find a single review,

Re: Issue 1949 in reviewboard: RBtools intermittently not submitting review requests.

Comment #2 on issue 1949 by mail.ax...@gmail.com: RBtools intermittently not submitting review requests. http://code.google.com/p/reviewboard/issues/detail?id=1949 I'm not sure whether the bad line is caused by the diff or something else as the commit usually goes through after a few

Issue 2039 in reviewboard: My Dashboard datagrid table is not properly aligned in Chrome browser (dev channel)

Status: New Owner: Labels: Type-Defect Priority-Medium New issue 2039 by chu.e...@gmail.com: My Dashboard datagrid table is not properly aligned in Chrome browser (dev channel) http://code.google.com/p/reviewboard/issues/detail?id=2039 * NOTE: Do not post confidential information in

Re: Issue 1633 in reviewboard: Repository configuration should not expose passwords

Comment #10 on issue 1633 by rtimush: Repository configuration should not expose passwords http://code.google.com/p/reviewboard/issues/detail?id=1633 Use https is not a solution for the original issue (the problem was that passwords were visible when user clicks the view source button).

Issue 2041 in reviewboard: Option to add more context to the main page of a review

Status: New Owner: Labels: Type-Enhancement Priority-Medium New issue 2041 by marc.san...@gmail.com: Option to add more context to the main page of a review http://code.google.com/p/reviewboard/issues/detail?id=2041 What version are you running? 1.5.4 What's the URL of the page this

Re: Issue 1983 in reviewboard: Include the server's time zone in the Web API

Comment #3 on issue 1983 by trowb...@gmail.com: Include the server's time zone in the Web API http://code.google.com/p/reviewboard/issues/detail?id=1983 You're right. I meant https://github.com/reviewboard/reviewboard/commit/171d857102ee6890f90916aa8276b85ad7a80453 -- You received this

Re: Issue 1983 in reviewboard: Include the server's time zone in the Web API

Comment #4 on issue 1983 by robert.munteanu: Include the server's time zone in the Web API http://code.google.com/p/reviewboard/issues/detail?id=1983 Great, thank you. -- You received this message because you are subscribed to the Google Groups reviewboard-issues group. To post to this

Re: Issue 2041 in reviewboard: Option to add more context to the main page of a review

Updates: Status: Confirmed Labels: Component-Reviews Comment #1 on issue 2041 by chip...@gmail.com: Option to add more context to the main page of a review http://code.google.com/p/reviewboard/issues/detail?id=2041 I've wanted this before too. It'll depend on the incremental