Code review comment for lp://qastaging/~kamstrup/libunity/merge-strategy

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

Ok, looks better, but please add a FIXME to the notifications/synchronizers that they should remove the models from their internal lists if the instance changes. (which will be completely common with private connections, might happen with remote models even now as well)

review: Approve

« Back to merge proposal