Code review comment for lp:~abentley/launchpad/attachment-timeout

Revision history for this message
Brad Crittenden (bac) wrote :

Aaron,

I've looked at the code changes thoroughly, and ran it locally to see how it behaved, and it looks good. I had a question about the placement of the download link but your rationale convinced me.

Thanks for clearing up my confusion about the page template.

--Brad

review: Approve (code)

« Back to merge proposal