Code review comment for lp:~michael-sheldon/content-hub/fix-1377334

Revision history for this message
Ken VanDine (ken-vandine) wrote :

Did you perform an exploratory manual test run of the code change and any related functionality on device or emulator?
 * Yes, tested on mako. I didn't reproduce the crash and verified no regressions when transfers are canceled with the dialog open.

Did CI run pass? If not, please explain why.
 * They haven't run yet, this branch shouldn't affect them.

Have you checked that submitter has accurately filled out the submitter checklist and has taken no shortcut?
 * Yes

review: Approve

« Back to merge proposal