search

OCA / account-budgeting

38 stars 113 forks source link

[12.0][ADD] account_budget_oca: migration analysis #53

Closed MiquelRForgeFlow closed 2 years ago

MiquelRForgeFlow commented 2 years ago

Follow up of https://github.com/OCA/OpenUpgrade/pull/2995.

cc: @pedrobaeza @legalsylvain

MiquelRForgeFlow commented 2 years ago

In fact, we could rename the one2many field crossovered_budget_line to crossovered_budget_line_ids with rename_fields(), but it's not strictly necessary. Or the theoritical_amount to theoretical_amount. But yes, we could say it's a "nothing to do". I don't bother to do any change.

OCA-git-bot commented 2 years ago

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖