Re: Summary column is shoing None after upgrading to RB 2.5

2015-11-03 Thread Sam
Hello, Is there any ETA as when we get 2.5.1? If not today, is there any workaround that I can apply? Regards, -- Supercharge your Review Board with Power Pack: https://www.reviewboard.org/powerpack/ Want us to host Review Board for you? Check out RBCommons: https://rbcommons.com/ Happy user?

RB 2.5 and Comment Categorization extension

2015-11-03 Thread Paul Wolf
I'm having problems getting this extension working on a new RB 2.5 installation. I'm able to install the extension, enable it, and configure some comment types. However, when I go to add a review comment, the Type 'select' has no values. I've tried restarting my server but that didn't help.

Re: Summary column is shoing None after upgrading to RB 2.5

2015-11-03 Thread Christian Hammond
It will be today. We're in California (PST timezone), for reference, but it should be out by early evening our time. Christian -- Christian Hammond - christ...@beanbaginc.com Review Board - https://www.reviewboard.org Beanbag, Inc. - https://www.beanbaginc.com On Tue, Nov 3, 2015 at 9:34 AM,

Re: Upgrade 2.0.20 to 2.5 failing on diffviewer_filediff table

2015-11-03 Thread Christian Hammond
Hi John, Sounds like there's an index it's trying to apply that already exists. What you can try doing is dropping that index in MySQL (diffviewer_filediff_4d02a56a), and seeing if that allows a fresh upgrade to proceed. Christian -- Christian Hammond - christ...@beanbaginc.com Review Board -

Re: What is the best way to get the status of the review request automatically

2015-11-03 Thread David Trowbridge
There's no way to get a response that only includes a single field's contents--it will always be formatted as JSON or XML. However, if you'd like to reduce the amount of data that gets sent, you can limit it using /api/review-requests/110/?only-fields=status -David On Mon, Nov 2, 2015 at 7:02

Enabling authentication extension

2015-11-03 Thread Graeme Perrow
I am trying to create an authentication extension for reviewboard (running 2.0.18). I've followed the authentication backend instructions here and the installation instructions here

Re: Upgrade 2.0.20 to 2.5 failing on diffviewer_filediff table

2015-11-03 Thread John McGowan
Thanks Christian, that did the trick. Had to run it for 2 indexes, affected 1/2 million rows in that table :) drop index diffviewer_filediff_4d02a56a on diffviewer_filediff; drop index diffviewer_filediff_79f4fcf6 on diffviewer_filediff; Just waiting on 2.5.1 now. Thanks again John On

Re: Enabling authentication extension

2015-11-03 Thread Graeme Perrow
That fixed it, thanks! Now I get invalid SQL being executed, but that's a different problem. Graeme On Tuesday, 3 November 2015 19:33:23 UTC-5, Christian Hammond wrote: > > The extension ID has now changed, due to the new location of the > extension. You'll probably end up with two entries

Re: Enabling authentication extension

2015-11-03 Thread Graeme Perrow
I moved the code into sapauth/extension/__init__.py, created an empty sapauth/__init__.py, and changed setup.py to: setup( name='sapauth', version='1.0', description='SAP Authentication backend for sareviews', author='Graeme Perrow', packages=['sapauth.extension'],

Re: Enabling authentication extension

2015-11-03 Thread Christian Hammond
The extension ID has now changed, due to the new location of the extension. You'll probably end up with two entries for this, and they may conflict. Try going into the database editor in the admin UI, clicking Registered Extensions, and deleting the old entry. Then re-run 'setup.py develop' or

Re: What is the best way to get the status of the review request automatically

2015-11-03 Thread Sam
Thanks for quick reply. I tried the following and it still return all the field rbt api-get http://reviewboard.com/api/review-requests/110/?onlyfield=status {"stat": "ok", "review_request": {"status": "pending", "last_updated": "2015-10-28T21:51:08Z", "target_people": [{"href":

Re: What is the best way to get the status of the review request automatically

2015-11-03 Thread Barret Rennie
Hi Sam, The syntax is "only-fields", not “onlyfield" Regards, Barret > On Nov 3, 2015, at 7:37 PM, Sam wrote: > > Thanks for quick reply. > I tried the following and it still return all the field > > > rbt api-get

Re: Enabling authentication extension

2015-11-03 Thread Christian Hammond
You'll need to move your module into a submodule of sap auth, like sapauth.extension. Christian On Tuesday, November 3, 2015, Graeme Perrow wrote: > > Here is setup.py: > > from reviewboard.extensions.packaging import setup > > setup( name='sapauth', >

Re: Posting NEW source files for review?

2015-11-03 Thread Joseph Henry
Unfortunately we are using CVS. -- Supercharge your Review Board with Power Pack: https://www.reviewboard.org/powerpack/ Want us to host Review Board for you? Check out RBCommons: https://rbcommons.com/ Happy user? Let us know! https://www.reviewboard.org/users/ --- You received this message

Re: Posting NEW source files for review?

2015-11-03 Thread Christian Hammond
That's fine. Just 'cvs add' the file and use RBTools to post, and it should work. Christian -- Christian Hammond - christ...@beanbaginc.com Review Board - https://www.reviewboard.org Beanbag, Inc. - https://www.beanbaginc.com On Tue, Nov 3, 2015 at 5:28 PM, Joseph Henry

Re: UnicodeDecodeError: 'ascii' codec can't decode byte 0xc2 in position 80769: ordinal not in range(128)

2015-11-03 Thread Sam
I really need an expert advise on this. I still see this failure when I try to post the review but when I try to diff the same version of the file in clearcase I dont see any issue. I upgraded the RBTools but did not help. Really appreciate your input on this? Traceback (most recent call

Re: Summary column is shoing None after upgrading to RB 2.5

2015-11-03 Thread Sam
Thanks a lot, RB 2.5.1 fixed this issue -- Supercharge your Review Board with Power Pack: https://www.reviewboard.org/powerpack/ Want us to host Review Board for you? Check out RBCommons: https://rbcommons.com/ Happy user? Let us know! https://www.reviewboard.org/users/ --- You received this

Re: Enabling authentication extension

2015-11-03 Thread Christian Hammond
Hi, I'd have to see the source of the extension to really be able to tell you more, but it sounds like perhaps the extension or its registration (in setup.py) isn't using absolute module paths to refer to some module. Would you be able to show me how you're registering it in setup.py, and the

Re: Enabling authentication extension

2015-11-03 Thread Graeme Perrow
Here is setup.py: from reviewboard.extensions.packaging import setup setup( name='sapauth', version='1.0', description='SAP Authentication backend for sareviews', author='Graeme Perrow', packages=['sapauth'], entry_points={ 'reviewboard.extensions':

Re: Review Board 2.5.1 is released

2015-11-03 Thread Christian Hammond
There was a breakage caused by the new post-commit review improvements, which broke a number of clients. We've temporarily reverted this and released 2.5.1.1, with plans to get that feature fixed up properly for 2.5.2. Sorry for the trouble! Christian -- Christian Hammond -