Code review comment for lp://staging/~therp-nl/ocb-server/6.1-lp933496

Revision history for this message
Lionel Sausin - Initiatives/Numérigraphe (ls-initiatives) wrote :

The patch can be further improved by using a TransientModel instead of a Model for testing. In v7.0 this fixes a warning about missing permissions. See http://bazaar.launchpad.net/~numerigraphe-team/ocb-server/7.0-po-targets-933496-vmt/revision/4798
I don't see any other problem than this detail, so I approve.

review: Approve

« Back to merge proposal