Code review comment for lp:~sylvain-pineau/checkbox/fix-1578579

Revision history for this message
Pierre Equoy (pieq) wrote :

I tested this patch on the device where lp:1578579 was found, and it does, indeed, fix the problem.

I thought it might introduce a regression with the job results history, but after testing it a bit more it seems OK and the whole history is kept even when the session is resumed.

+1, thanks a lot!

review: Approve

« Back to merge proposal