openerp-community-reviewer team mailing list archive

[Lorenzo Battistini - Agile BG <lorenzo.battistini@xxxxxxxxxxx>]

On 09/25/2013 12:46 AM, Pedro Manuel Baeza wrote:
> - Use invoice_webkit as dependency. It introduces a higher dependency (webkit) that is not needed by the funcionality itself. There are a lot of people that don't still use webkit.

Hello Pedro,

the main purpose of the module is to print the invoice with lots 
information (just changed a bit the module description). So, a report 
engine is needed.
Maybe we could move it to 
lp:~account-core-editors/account-invoice-report/7.0 branch


> - Load data into a text field. Why don't load it in a many2many field like other references (order_lines and so on)?

Data is already available within a (function) m2m field: 'prod_lot_ids'.
Loading data into 'note' is for printing purpose only.

-- 
https://code.launchpad.net/~agilebg/account-invoicing/adding_account_invoice_production_lot_7/+merge/181747
Your team Account Core Editors is subscribed to branch lp:account-invoicing.