Thanks for the patch, but I'm pretty concerned about monkey-patching request.user like that, as it could interfere with other bits of code. I'd rather have proper support for passing a desired user (defaulting to request.user) through the dashboard code.
Supporting dashboard impersonations
Review Request #3207 - Created July 12, 2012 and discarded
Around a year back I made a little hack so that administrators could see what users do on their dashboard via a 'user' parameter. This has come in handy for diagnosing issues on several occasions so checking if you would like to have it upstream. This is available in my 'dashboard_impersonation' branch... https://github.com/atagar/ReviewBoard/tree/dashboard_impersonation
We've been running this change with ReviewBoard 1.5, and it hasn't been exercised with 1.6. To run it simply provide a user argument such as... http://reviews.reviewboard.org/dashboard/?user=atagar
Loading file attachments...
I tested this on (1.7 RC1) and it works as described. I noticed that there was disagreement of using this patch, after having tested it. Though I'd show my input.