savoirfairelinux-openerp team mailing list archive

[Virgil Dupras <virgil.dupras@xxxxxxxxxxxxxxxxxxxx>]

Review: Approve

I reviewed the module in a summarily manner (not being familiar with the purpose of the module):

1. Re-produced the inability to install of the trunk branch under 7.0
2. Installed the MP successfully under 7.0
3. Created a claim with a Return line, which seemed to work without error.

I don't see the point in commenting code instead of removing it, but otherwise, the change looks clean.
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-rma/7.0-port-merged/+merge/177975
Your team Savoir-faire Linux' OpenERP is subscribed to branch lp:~savoirfairelinux-openerp/openerp-rma/7.0-port-merged.