On 06/02/2014 01:05 PM, Sean Dague wrote: > On 06/02/2014 09:21 AM, Matthew Treinish wrote: > <snip> >>> The url for this is - http://goo.gl/g4aMjM >>> >>> (the long url is very long: >>> https://review.openstack.org/#/dashboard/?foreach=%28project%3Aopenstack%2Ftempest+OR+project%3Aopenstack-dev%2Fgrenade+OR+project%3Aopenstack%2Fqa-specs%29+status%3Aopen+NOT+owner%3Aself+NOT+label%3AWorkflow%3C%3D-1+label%3AVerified%3E%3D1%2Cjenkins+NOT+label%3ACode-Review%3C%3D-1%2Cself+NOT+label%3ACode-Review%3E%3D1%2Cself&title=QA+Review+Inbox&QA+Specs=project%3Aopenstack%2Fqa-specs&Needs+Feedback+%28Changes+older+than+5+days+that+have+not+been+reviewed+by+anyone%29=NOT+label%3ACode-Review%3C%3D2+age%3A5d&Your+are+a+reviewer%2C+but+haven%27t+voted+in+the+current+revision=reviewer%3Aself&Needs+final+%2B2=%28project%3Aopenstack%2Ftempest+OR+project%3Aopenstack-dev%2Fgrenade%29+label%3ACode-Review%3E%3D2+limit%3A50&Passed+Jenkins%2C+No+Negative+Feedback=NOT+label%3ACode-Review%3E%3D2+NOT+label%3ACode-Review%3C%3D-1+limit%3A50&Wayward+Changes+%28Changes+with+no+code+review+in+the+last+2days%29=NOT+label%3ACode-Review%3C%3D2+age%3A2d >>> >>> The url can be regenerated easily using the gerrit-dash-creator. >>> >> >> These generated URLs don't quite work as expected for me, I see a bunch of -1s >> from jenkins in all the sections. Other things like reviews with -2s showing up >> "in need final +2", or reviews with -2s and +2s from me being listed in the "but >> haven't voted in the current revision". Also the top section just seems to list >> every open QA program review regardless of it's current review state. >> >> I'll take a look at the code and see if I can help figure out what's going on. > > It appears that there is some issue in Firefox vs. Gerrit here where > Firefox is incorrectly over unescaping the URL, thus it doesn't work. > Chrome works fine. As I'm on Linux that's the extent of what I can > natively test. > > I filed a Firefox bug here - > https://bugzilla.mozilla.org/show_bug.cgi?id=1019073 The following updated url seems to work for firefox: https://goo.gl/oGYH4s Thanks to dtantsur for figuring out the extra escaping on commas you needed to work with firefox. -Sean -- Sean Dague http://dague.net -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 482 bytes Desc: OpenPGP digital signature URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20140602/f95044bb/attachment.pgp>