[REF] stock_move_entries, invoice_discount, commission_payment: Improvement performance on computed fields

Vauxoo / addons-vauxoo

All our modules related to developments that solves generic issues on Odoo, or that solve internal problems on Odoo Core, if something is here, maybe it is solving an issue in your company, try it and report what you see.

http://www.vauxoo.com

193 stars 288 forks source link

[REF] stock_move_entries, invoice_discount, commission_payment: Improvement performance on computed fields #1219

Closed moylop260 closed 7 years ago

moylop260 commented 7 years ago

Add api.depends and store=True to computed fields and migrated to new api

Dummy PR: https://github.com/Vauxoo/lodigroup/pull/719

moylop260 commented 7 years ago

@josemoralesp @luistorresm Could you give me your technical revision, please?

moylop260 commented 7 years ago

@luisg123v Could you help me to check if migrations script is working fine vs the original behaviour?

hbto commented 7 years ago

@moylop260 Please rebase your branch on new commits from https://github.com/Vauxoo/addons-vauxoo/pull/1218

Regards

hbto commented 7 years ago

I would like to have these changes stabilized in order to make them available when https://github.com/Vauxoo/addons-vauxoo/pull/1218 get merged

@josemoralesp @moylop260 @luisg123v @luistorresm

moylop260 commented 7 years ago

LGTM

hbto commented 7 years ago

LGTM

luisg123v commented 7 years ago

I've tested the fields from the invoice model, they're working as expected.