Matthieu Dietrich @ camptocamp has proposed merging
lp:~camptocamp/account-invoicing/7.0-update-invoice-view-mdh into
lp:account-invoicing.
Requested reviews:
Account Core Editors (account-core-editors)
For more details, see:
https://code.launchpad.net/~camptocamp/account-invoicing/7.0-update
The proposal to merge
lp:~camptocamp/account-invoicing/7.0-update-invoice-view-mdh into
lp:account-invoicing has been updated.
Commit Message changed to:
[FIX] update invoice form view to be conforming of the addons' one
For more details, see:
https://code.launchpad.net/~camptocamp/account-inv
The proposal to merge
lp:~camptocamp/account-invoicing/7.0-update-invoice-view-mdh into
lp:account-invoicing has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~camptocamp/account-invoicing/7.0-update-invoice-view-mdh/+merge/196835
--
https
The proposal to merge
lp:~camptocamp/account-invoicing/7.0-update-invoice-view-mdh into
lp:account-invoicing has been updated.
Status: Approved => Merged
For more details, see:
https://code.launchpad.net/~camptocamp/account-invoicing/7.0-update-invoice-view-mdh/+merge/196835
--
https://cod
Review: Approve code review, no tests
LGTM, Thanks for the fix
--
https://code.launchpad.net/~camptocamp/account-invoicing/7.0-update-invoice-view-mdh/+merge/196835
Your team Account Core Editors is subscribed to branch lp:account-invoicing.
--
Mailing list: https://launchpad.net/~openerp-commu
Review: Needs Information
Hi, Matthieu,
Thanks for the MP. At first instance, I don't understand why the view has been
overwriten entirely. Can you please tell me?
Regards.
--
https://code.launchpad.net/~camptocamp/account-invoicing/7.0-update-invoice-view-mdh/+merge/196835
Your team Account C
Review: Approve code review, no test
Hello,
Pedro I do not get the meaning of your question.
Do you mean why they replace the whole header tag?
else for the current MP it looks good to me
Regards Nicolas
--
https://code.launchpad.net/~camptocamp/account-invoicing/7.0-update-invoice-view-mdh/+
Yeah, that is! It's not something explicit of this MP, but I have detected
checking it.
Regards.
--
https://code.launchpad.net/~camptocamp/account-invoicing/7.0-update-invoice-view-mdh/+merge/196835
Your team Account Core Editors is subscribed to branch lp:account-invoicing.
--
Mailing list: h
Review: Approve code review
--
https://code.launchpad.net/~camptocamp/account-invoicing/7.0-update-invoice-view-mdh/+merge/196835
Your team Account Core Editors is subscribed to branch lp:account-invoicing.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : opener
9 matches
Mail list logo