Code review comment for lp:~dev-nigelj/launchpad/bug-810551

Revision history for this message
Gavin Panella (allenap) wrote :

This is a nice improvement. You could even go further and remove the leading date from the titles on +announcements, but this is good on its own.

It does, however, need a test, even though there wasn't one before. I haven't done any coding in a while, and found myself too tempted to resist writing one myself - http://paste.ubuntu.com/679849/ - if you'd like to use it.

review: Needs Fixing

« Back to merge proposal