[14.0][FIX] l10n_it_fiscal_document_type get only possible fiscal document type #4340
Conversation
sergiocorato
force-pushed
the
14.0-fix-l10n_it_fiscal_document_type-get-only-applicable-document-type
branch
3 times, most recently
from
1752bd3
to
d32e238
Compare
There was a problem hiding this comment.
Grazie della PR!
Ho verificato che il problema esiste in 14.0 ed è risolto da queste modifiche.
Potresti aggiungere una nota nell'help del campo account.fiscal.position.fiscal_document_type_id per indicare come mai quella posizione fiscale non verrà sempre applicata come succedeva prima?
sergiocorato
force-pushed
the
14.0-fix-l10n_it_fiscal_document_type-get-only-applicable-document-type
branch
3 times, most recently
from
de24389
to
20a883c
Compare
| # Create a customer invoice for a partner with a fiscal position with TD10, | ||
| # which is not applicable for "out_invoice" so it's applied the default TD01 |
There was a problem hiding this comment.
Questo caso è testato anche sotto nell'altro test: se ne può rimuovere uno? Altrimenti puoi evidenziare che differenza c'è?
| # Create a vendor invoice for a partner with a fiscal position with TD10, | ||
| # which is applied as applicable for "in_invoice". |
There was a problem hiding this comment.
Questo caso è testato anche sotto nell'altro test: se ne può rimuovere uno? Altrimenti puoi evidenziare che differenza c'è?
There was a problem hiding this comment.
Ah vero, lo avevo messo sotto poi, grazie.
sergiocorato
force-pushed
the
14.0-fix-l10n_it_fiscal_document_type-get-only-applicable-document-type
branch
from
20a883c
to
71c3b1f
Compare
There was a problem hiding this comment.
L'ho notato solo ora con l'ultimo push, non so se era così anche prima, ma potresti modificare il messaggio del commit in modo che segua le linee guida https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#71commit-message?
In particolare
please check if the commit message is cut with ellipsis
Ho inoltre corretto un test in cui l'intenzione credo fosse di creare delle fatture e non dei movimenti contabili.