OCA / l10n-italy

Odoo Italian localization
https://www.odoo-italia.org
GNU Affero General Public License v3.0
154 stars 306 forks source link

IMP l10n_it_account removing check_balance_sign_coherence. #4431

Open eLBati opened 3 weeks ago

eLBati commented 3 weeks ago

This check can block chart of accounts configuration and is not mandatory. account_balance_sign is only used by l10n_it_financial_statements_report and, in case of incoherent signs, user can visualize the amounts in financial_statements_report and fix the sign, in case

eLBati commented 2 weeks ago

pre-commit verde

eLBati commented 1 week ago

I think this is at least unexpected

Mi pare il comportamento atteso invece, come mai non dovrebbe?

Could you please write in the commit the reason of this change instead of just describing it?

Esteso

SirAionTech commented 1 week ago

I think this is at least unexpected

Mi pare il comportamento atteso invece, come mai non dovrebbe?

Nel gruppo di conti c'è un campo segno, nei singoli conti all'interno del gruppo c'è un campo segno; visto che sono oggetti imparentati mi aspetto che ci sia una specie di ereditarietà: se un gruppo ha un segno allora anche tutti i conti nel gruppo hanno lo stesso segno.

Se è il comportamento atteso da questa PR lo puoi chiarire? Sfido qualsiasi funzionale a capire che

removing check_balance_sign_coherence

o

This check can block chart of accounts configuration and is not mandatory. account_balance_sign is only used by l10n_it_financial_statements_report and, in case of incoherent signs, user can visualize the amounts in financial_statements_report and fix the sign, in case

Vuol dire che i conti all'interno di un gruppo possono avere segno diverso dal gruppo


P.S. ma quindi inglese o italiano?