Code review comment for lp:~bhdouglass/rockwork/translations

Revision history for this message
Michael Zanetti (mzanetti) wrote :

Ok. merged it. There was a bug. The installed apps/watchfaces model was broken. Apparently dynamicRoles: true only works properly if the first inserted item has all the roles. In our case, the showWatchApps and showWatchFaces properties are not in the first one so those broke. Anyhow, I've fixed that before merging.

review: Approve

« Back to merge proposal