Code review comment for lp://staging/~abreu-alexandre/oxide/file-picker-done-guard

Revision history for this message
Chris Coulson (chrisccoulson) wrote :

> > Haven't I already reviewed this change with a request to add a test for it?
>
> Did you already? ... I couldn't find another branch for that,
>
> I'll add a test,

It was your branch :)

The review URL was https://code.launchpad.net/~abreu-alexandre/oxide/filepicker-accept-nullcheck/+merge/264833, but it seems to have disappeared now. I approved it around July last year with a comment that it should be committed with a test.

« Back to merge proposal