[ADD]l10n_it_edi_doi_extension: modulo per gestione dichiarazioni di intento emesse dall'azienda#4680
Conversation
matteotognini
force-pushed
the
18.0-add-l10n_it_doi_oca
branch
from
5d76f14 to
7edd183
Compare
|
Questo modulo risulta un'estensione di Il suffisso Se vogliamo mantenere lo standard attuale direi che il nome andrebbe cambiato in |
matteotognini
force-pushed
the
18.0-add-l10n_it_doi_oca
branch
from
7edd183 to
49d1694
Compare
matteotognini
changed the title
|
@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 |
There was a problem hiding this comment.
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?
matteotognini
force-pushed
the
18.0-add-l10n_it_doi_oca
branch
from
2ce09cf to
4afa0a5
Compare
monen17
left a comment
monen17
left a comment
There was a problem hiding this comment.
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?
matteotognini
force-pushed
the
18.0-add-l10n_it_doi_oca
branch
from
4afa0a5 to
d0cd3ed
Compare
monen17
left a comment
monen17
left a comment
There was a problem hiding this comment.
Grazie mille delle modifiche!
Ora l'ho guardata meglio, vedi i commenti qui sotto
matteotognini
force-pushed
the
18.0-add-l10n_it_doi_oca
branch
2 times, most recently
from
f8c2132 to
309dde2
Compare
monen17
left a comment
monen17
left a comment
There was a problem hiding this comment.
Grazie delle modifiche!
A livello di codice per me siamo quasi a posto, iniziamo a fare qualche revisione funzionale? @odooNextev
matteotognini
force-pushed
the
18.0-add-l10n_it_doi_oca
branch
from
309dde2 to
5056988
Compare
VBNext
left a comment
VBNext
left a comment
There was a problem hiding this comment.
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
matteotognini
force-pushed
the
18.0-add-l10n_it_doi_oca
branch
2 times, most recently
from
7adddf9 to
b9a6933
Compare
monen17
left a comment
monen17
left a comment
There was a problem hiding this comment.
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.
matteotognini
force-pushed
the
18.0-add-l10n_it_doi_oca
branch
from
b9a6933 to
402b16d
Compare
matteotognini
changed the title
matteotognini
force-pushed
the
18.0-add-l10n_it_doi_oca
branch
from
402b16d to
294892a
Compare
|
/ocabot merge nobump |
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at 2a403d9. Thanks a lot for contributing to OCA. ❤️ |
8 participants
No description provided.