account-financial-tools icon indicating copy to clipboard operation
account-financial-tools copied to clipboard

Migration to version 14.0

Open OCA-git-bot opened this issue 3 years ago • 32 comments

Todo

https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-14.0

Modules to migrate

  • [x] account_asset_management - By @joao-p-marques - #1112
  • [x] account_balance_line - By @hildickethan - #1105
  • [x] account_chart_update - By @Reyes4711 - #1136
  • [ ] account_check_deposit
  • [ ] account_fiscal_year: discussion in #1069
  • [ ] account_loan - By @AntoniRomera - #1316
  • [ ] account_lock_date_update
  • [ ] account_lock_to_date - By @AntoniRomera - #1314
  • [ ] account_menu
  • [ ] account_move_budget - By @AntoniRomera - #1315
  • [ ] account_move_fiscal_year - By @xavier-bouquiaux - #1263
  • [ ] account_move_line_purchase_info
  • [ ] account_move_line_tax_editable
  • [ ] account_move_reversal_choose_method - By @AlvaroTForgeFlow - #1278
  • [ ] account_move_template
  • [ ] account_netting
  • [ ] account_spread_cost_revenue - By @kittiu - #1244
  • [ ] account_tax_repartition_line_tax_group_account
  • [ ] base_vat_optional_vies
  • [x] account_move_force_removal: By @tafaRU - https://github.com/OCA/account-financial-tools/pull/1167

Missing module? Check https://github.com/OCA/maintainer-tools/wiki/%5BFAQ%5D-Missing-modules-in-migration-issue-list

OCA-git-bot avatar Oct 04 '20 10:10 OCA-git-bot

@pedrobaeza Since the reconciliation is not available in the v14.0, do you think we can have more detailed account_netting module, we have a PR in here https://github.com/OCA/l10n-romania/pull/76, which basically does the same, allow you to select the amounts (partial, full) and add notes for detailing the compensation.

feketemihai avatar Oct 12 '20 19:10 feketemihai

Well, the idea is to restore the reconciliation in an OCA module. We can include the compensation/netting there or continue with current module. What are the differences between account_netting and yours?

pedrobaeza avatar Oct 12 '20 19:10 pedrobaeza

@pedrobaeza first it adds separate models to keep track of compensations, compensation lines, amount that you want to distribute.

feketemihai avatar Oct 12 '20 19:10 feketemihai

Uhm, not sure about the convenience of having such overhead for a simple operation like this one.

pedrobaeza avatar Oct 12 '20 22:10 pedrobaeza

@pedrobaeza It's coming from the issues with Fiscal Authorities, since you need to document the compensation operation.

feketemihai avatar Oct 13 '20 06:10 feketemihai

Well, the reconciliation record itself is such document IMO.

pedrobaeza avatar Oct 13 '20 07:10 pedrobaeza

So, for account_fiscal_year we follow this approach https://github.com/OCA/account-financial-tools/issues/1069#issuecomment-706683943 right?

eLBati avatar Oct 23 '20 15:10 eLBati

I would say yes.

pedrobaeza avatar Oct 23 '20 15:10 pedrobaeza

I'm starting to work on the account_fiscal_year migration, I'll try to follow the guidelines given in https://github.com/OCA/account-financial-tools/issues/1069#issuecomment-706683943 and will create a PR as soon as I've got something working

SimoRubi avatar Nov 05 '20 15:11 SimoRubi

Thanks, Simone. Ping me with any doubt.

pedrobaeza avatar Nov 05 '20 15:11 pedrobaeza

account_check_deposit https://github.com/OCA/account-financial-tools/pull/1082

Saran440 avatar Nov 09 '20 06:11 Saran440

account_fiscal_year: https://github.com/OCA/account-financial-tools/pull/1081

SimoRubi avatar Nov 10 '20 08:11 SimoRubi

On the road to restore reconciliation widget. I would like to know if it fits in this OCA repo as 'account_reconcile_widget'. Comments and indications also woud be appreciated.

ozono avatar Nov 20 '20 10:11 ozono

@ozono I have that on my development backlog, talked on OCA Online Days. Check https://twitter.com/alexisdlattre/status/1317149095597887491?s=20

Do you want us to meet for debating about this?

pedrobaeza avatar Nov 20 '20 10:11 pedrobaeza

@ozono I have that on my development backlog, talked on OCA Online Days. Check https://twitter.com/alexisdlattre/status/1317149095597887491?s=20

Do you want us to meet for debating about this?

Ok Pedro. I'll send you a mail to make an appointment.

ozono avatar Nov 20 '20 10:11 ozono

Working on account_reconciliation_widget #1101

ozono avatar Dec 15 '20 22:12 ozono

account_reconciliation_widget development was moved to from this repo to OCA/account-reconcile#359 after request

ozono avatar Dec 17 '20 00:12 ozono

I will migrate account_asset_management

joao-p-marques avatar Jan 07 '21 14:01 joao-p-marques

please add: account_invoice_currency to the list

bosd avatar Jan 25 '21 13:01 bosd

account_reversal features are included in core.

pedrobaeza avatar Mar 23 '21 15:03 pedrobaeza

account_move_force_removal

Please, add account_move_force_removal in the modules to migrate list

Thanks

fgarcia-humanoide avatar Apr 13 '21 05:04 fgarcia-humanoide

I'm starting to work on account_move_force_removal migration.

tafaRU avatar Apr 26 '21 08:04 tafaRU

Please, add account_move_force_removal in the modules to migrate list

@fgarcia-humanoide done

tafaRU avatar Apr 26 '21 08:04 tafaRU

account_move_force_removal: https://github.com/OCA/account-financial-tools/pull/1167

tafaRU avatar Apr 26 '21 09:04 tafaRU

I'm starting to work on account_move_force_removal migration.

Thank you very much Alex - @tafaRU

fgarcia-humanoide avatar May 05 '21 06:05 fgarcia-humanoide

FYI, I'm working on porting account_permanent_lock_date and account_permanent_lock_move_update to v14. I think this feature is important. Any serious auditor would be very surprised to see that you can update lock dates backwards without any problem!

In v13/v14, there are 3 native lock date fields: https://github.com/odoo/odoo/blob/14.0/addons/account/models/company.py#L43 period_lock_date, fiscalyear_lock_date and tax_lock_date. The module account_permanent_lock_date adds a new field permanent_lock_date. So, that would be a total of 4 lock dates... it's starting to be very difficult for users to understand the difference between these 4 lock dates! So I propose the following: the module account_permanent_lock_date would use the native "fiscalyear_lock_date" and make sure it cannot be set backwards or set to null.

Advantages:

  1. no need to inherit create() and write() of account.move in account_permanent_lock_date => perf improvement and code simplification
  2. no need to maintain the "glue" module account_permanent_lock_move_update
  3. only 3 lock dates fields instead of 4 => easier to understand for accountants.

So I plan to implement this change in the migration PR of the module account_permanent_lock_date. What do you think ?

alexis-via avatar Nov 18 '21 18:11 alexis-via

Although I don't use such module (at least not for now, there's a new law incoming that I think will force us to use it), I see it correct and a better implementation.

pedrobaeza avatar Nov 18 '21 18:11 pedrobaeza

Thanks for sharing your thoughts Pedro! BTW, as I will not re-use any existing line of code of account_permanent_lock_move, it will not be a "migration" but a new module. As it's a new module, it could have a new name (for example: account_fiscalyear_lock_date_no_backwards) My opinion: I prefer to keep the current name because the feature is similar. But, if some people prefer to keep the module account_permanent_lock_date with the field "permanent_lock_date", then I'll propose my module under a new name.

alexis-via avatar Nov 18 '21 18:11 alexis-via

+1 for keeping the name.

pedrobaeza avatar Nov 18 '21 18:11 pedrobaeza