Code review comment for lp:~bac/launchpad/bug-799901

Revision history for this message
Данило Шеган (danilo) wrote :

Another minor note from me (I've had the tab loaded, sorry :)): perhaps you can also move the display_subscribed_by into the view and not rely on storm caching since you've already have both subscriber and subscribed_by objects which you can directly compare?

Otherwise, looks great!

« Back to merge proposal