Code review comment for lp:~camptocamp/stock-logistic-barcode/7.0-fix_1330459-sge+afe

Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote :

When you want to rename a field, you can also use the 'oldname' argument, example in crm:

        'opt_out': fields.boolean('Opt-Out', oldname='optout',
            help="If opt-out is checked, this contact has refused to receive emails for mass mailing and marketing campaign. "
                    "Filter 'Available for Mass Mailing' allows users to filter the leads when performing mass mailing."),

The ORM would automatically rename the field.

Your version is also a valid way to rename the field so LGTM.

review: Approve (code review)

« Back to merge proposal