Launchpad itself

Code review comment for lp:~gmb/launchpad/bug-999554

  1. bug-999554
  2. Merge into devel
Revision history for this message
Francesco Banconi (frankban) wrote :

Graham, your branch looks good, thank you.
I've just spotted a problem: at line 8 of the diff you removed an interface previously exposed through the webservice.
This generate an error during AppServerLayer set up, e.g. running::

    bin/test -t lp.bugs.model.tests.test_bugtask.TestWebservice

Approved with that fix. Waiting for rvba approval.

review: Approve (code*)

« Back to merge proposal