Great job Guewen, I just read the code and I have some comment
Regarding the default value for the workflow method before it was 'creating 
invoice after delivery' now it's manual, maybe it' will be better to keep the 
same behaviour? And setting invoice => 'after delivery' instead of "manual".
also I am not sure that activating by default the option 
"invoice_date_is_order_date" is a good idea. It's usefull for importing an 
historic but in day to day you will have invoice number than will not match 
with the invoice date.

What do you think?
-- 
https://code.launchpad.net/~extra-addons-commiter/e-commerce-addons/7.0-sale-workflow-create_invoice_on/+merge/159321
Your team extra-addons-commiter is subscribed to branch 
lp:~extra-addons-commiter/e-commerce-addons/7.0-sale-workflow.

-- 
Mailing list: https://launchpad.net/~savoirfairelinux-openerp
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~savoirfairelinux-openerp
More help   : https://help.launchpad.net/ListHelp

Reply via email to