Launchpad itself

Code review comment for lp:~adiroiban/launchpad/bug-359180-take-2

  1. bug-359180-take-2
  2. Merge into devel
Revision history for this message
Graham Binns (gmb) wrote :

Hi Adi,

I really like this branch; good job! I'm not entirely happy about the lack of Windmill tests, but I understand what you mean about it being non-trivial to write one for this case. Please file a bug about adding some Windmill tests for this, detailing the difficulties you had. It might be worth mailing the launchpad-dev list to see if anyone has any suggestions.

review: Approve (code, js)

« Back to merge proposal