[ADD]l10n_it_edi_doi_extension: modulo per gestione dichiarazioni di intento emesse dall'azienda

[matteonext]

No description provided.

[primes2h]

Questo modulo risulta un'estensione di l10n_it_edi_doi.

Il suffisso _oca in genere viene utilizzato per moduli alternativi a quelli ufficiali.
Vedi ad es. #4649 e la discussione relativa a l10n_it_edi_extension https://discord.com/channels/753902328494424064/1200363648011477063/1235859547193737239

Se vogliamo mantenere lo standard attuale direi che il nome andrebbe cambiato in l10n_it_edi_doi_extension.

[francesco-ooops]

@matteonext se nel modello che aggiungi c'è il chatter possiamo indicare la issue #4671 come fatta in questa PR per la 18

[matteonext]

@matteonext se nel modello che aggiungi c'è il chatter possiamo indicare la issue #4671 come fatta in questa PR per la 18

@francesco-ooops non aggiungo modelli, nel modello è della dichiarazione d'intento è già presente il chatter. Credo che puoi indicare come fatto in quella issue

[monen17]

Grazie della PR!
Puoi modificare il messaggio del commit in modo che segua https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#71commit-message? In particolare il primo punto è

Commit messages are in English

Se ti serve spazio nel titolo del commit, puoi accorciare il nome del modulo in l10n_it_edi_doi_ext.

Puoi aggiungere almeno un test?

[monen17]

Valgono ancora i punti che scrissi settimana scorsa in #4680 (review):

Puoi modificare il messaggio del commit in modo che segua https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#71commit-message? In particolare il primo punto è

Commit messages are in English

Se ti serve spazio nel titolo del commit, puoi accorciare il nome del modulo in l10n_it_edi_doi_ext.

Puoi aggiungere almeno un test?

Inoltre, un problema generale mi pare sia che viene spesso rotta l'ereditarietà dei metodi, ne ho segnato qualcuno dove andrebbe chiamato super ma credo ce ne siano anche altri.
Potresti correggere?

[monen17]

Grazie mille delle modifiche!
Ora l'ho guardata meglio, vedi i commenti qui sotto

[monen17]

Grazie delle modifiche!
A livello di codice per me siamo quasi a posto, iniziamo a fare qualche revisione funzionale? @odooNextev

[VBNext]

Funziona correttamente.
Note:
1- non è più necessario avere la posizione fiscale dichiarazione d'intento, se non per la mappatura delle imposte
2- Da modificare la selezione della dichiarazione di intento nelle fatture filtrando per tipo (es. fatture cliente - devo vedere solo DOI ricevute)
3- valutare se segnalare ad Odoo il fatto che sia possibile superare l'importo della DOI

[monen17]

Oltre a un vecchio punto aperto che non ha ancora ricevuto risposta, ho solo aggiunto qualche commento sul nuovo (nell'ultima revisione non c'era) campo account.move.doi_type.

[eLBati]

/ocabot merge nobump

[OCA-git-bot]

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-4680-by-eLBati-bump-nobump, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at 2a403d9. Thanks a lot for contributing to OCA. ❤️