Code review comment for lp://staging/~elachuni/ubuntu-webcatalog/celery

Revision history for this message
James Westby (james-w) wrote :

30 -openid_launchpad_staff_teams = canonical-ca-hackers
31 +openid_launchpad_staff_teams = canonical-isd-hackers

That looks backwards?

332 + if (message.hasClass('success')) {

That looks as if it will loop forever on successful tasks?

Most of this branch looks pretty good to me.

Thanks,

James

« Back to merge proposal