Code review comment for lp:~edwin-grubbs/launchpad/bug-577492-packaging-duplicates

Revision history for this message
Edwin Grubbs (edwin-grubbs) wrote :

> Looks good.  Q: should you be using the slave store instead of IStore(self)?

IStore() uses the default store, so it will use the slave store
automatically unless the user has made writes on launchpad recently,
in which case, using the master store is necessary to ensure that it
doesn't appear that the user's changes got reverted.

« Back to merge proposal