this should outline my huge change :)

Thanks for the patch! Generally with a person's first few patches, we review fairly heavily, since there's a lot of things people may not be familiar wiht in terms of Django, our codebase, and our coding style, so I'll point them all out here.

There's a terminology difference here. It really should be "Review Request" and not "Review".

Also, variables should be {{foo}} rather than {{ foo }} or {{ foo}}. I realize that the we have {{ lastupdated}} in there already, but that's a typo.

One more thing, which is just one of those things that you learn as you learn Django. Rather than hard-coding paths to pages, you should be using {{review_request.get_absolute_url}}. This will return the proper path even if the page moves locations (or if it's installed in a subdirectory).

Hey - there's still some terminology confusion here:

reviewid should be something like 
reviewrequestid, and reviewurl should be something like reviewrequesturl.

Or maybe review_request_id and review_request_url for readability.

Thanks again for the patch!

One last tiny suggestion - purely for code beautification.

-Mike

Last suggestion:

Only 1 space between "blocktrans" and "with", and then maybe put the contents of the block on different lines, like:

{% blocktrans with ...yaddayadda %}
  Review Request <a href="yadda yadda"...
{% endblocktrans %}

That's all I've got though. Everything else looks OK.  Once this is done, you've got my ship-it.  :D

Looks good to me!  Thanks,

-Mike