Code review comment for lp://qastaging/~bilalakhtar/unity/sc-integration-phase2

Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

This branche introduces a change in behavior (flying icon). It clearly needs an UIFe and a FFe.
Also, we saw that the previous code was broken without anyone noticing. This shows that you need tests for you code. Can you please first ensure that lp:~bilalakhtar/unity/fix-sc-launcher-integration gets autopilot tests for it not regressing again?

« Back to merge proposal