Code review comment for lp://qastaging/~stolowski/unity-scope-home/preview-on-lmb

Revision history for this message
Michal Hruby (mhr3) wrote :

Looks reasonable, I don't think the "yield handle_metrics ();" should be right after "yield handle_preview ()", but as we discussed this will be fixed in another branch.

review: Approve

« Back to merge proposal