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

[MIG] internal_transfer_with_agreed_amount: Migrate module from v12.0 to v16.0 T#71892 #1623

Closed CLaurelB closed 1 year ago

CLaurelB commented 1 year ago
CLaurelB commented 1 year ago

@luisg123v could you review, please?

luisg123v commented 1 year ago

@CLaurelB make sure everything is green before asking for review.

rolandojduartem commented 1 year ago

Hi, @luisg123v and @desdelinux, please, could you check if this is the correct approach?

rolandojduartem commented 1 year ago

Hi, @hbto, please, could you review this MR?

Branch to test: https://www.odoo.sh/project/rsfcr/branches/16.0-rsf-internal_transfer-t71892-rolando/history

rolandojduartem commented 1 year ago

Finally, this is ready. Please, @luisg123v, @hbto and @desdelinux, could you review this MR?

Branch: https://www.odoo.sh/project/rsfcr/branches/16.0-rsf-internal_transfer-t71892-rolando/history

hbto commented 1 year ago

@rolandojduartem @luisg123v @desdelinux

Looks good to me.

Best Regards.

desdelinux commented 1 year ago

@luisg123v Please, could you review?

rolandojduartem commented 1 year ago

@luisg123v friendly reminder

rolandojduartem commented 1 year ago

@luisg123v friendly reminder

desdelinux commented 1 year ago

@hugho-ad Could you review and merged, please :)

hbto commented 1 year ago

@rolandojduartem There are three pending comments that needing to be taken care.

Resolving this issue let us merge.

cc/ @hugho-ad @desdelinux @luisg123v

rolandojduartem commented 1 year ago

Hi, @luisg123v, please, could you review again?