Se sei d'accordo, in questa MP includerei solamente le modifiche atte a far funzionare il modulo stabile (quello sul branch ufficiale 6.1) sulla 7.0. Le correzioni ai bachi o eventuali miglioramenti al modulo le introdurrei tramite apposite merge proposals.
l 72-127: sono campi/modifiche che verrebbero introdotte da account_due_list, quindi, se lasciamo la dipendenza, non sono necessari
l 207: è forse una modifica che risolverebbe qualche bug conosciuto? La terrei comunque fuori per ora
l 311, 319, 328, 336, 631: come mai viene passato il self alla chiamata ad un metodo?
l 353, 728: anche questa riga risolve forse un bug esistente?
l 440: la scriverei così: creditor_province = creditor_address.province and creditor_address.province.code or ''
l 649, 658: è necessario per il porting alla 7?
l 745-750: probabilmente un enhancement da tenere fuori da questa MP
PS: non eliminare questa MP in caso ne volessi fare un'altra, al massimo la impostiamo a rejected. Ti consiglio comunque di fare le modifiche direttamente su lp:~enlightx/openobject-italia/7.0-l10n_it_ricevute_bancarie e questa MP verrà aggiornata automaticamente.
PPS: se hai bisogno di verificare il diff tra versione 6.1 e 7.0, puoi copiare nel branch 6.1 il modulo dal branch 7.0 e fare bzr diff
Se sei d'accordo, in questa MP includerei solamente le modifiche atte a far funzionare il modulo stabile (quello sul branch ufficiale 6.1) sulla 7.0. Le correzioni ai bachi o eventuali miglioramenti al modulo le introdurrei tramite apposite merge proposals.
Quindi, facendo riferimento al diff:
l 17: Lascerei la dipendenza da account_due_list. Per velocizzare il processo di merge di tale modulo sui branch community, invito tutti a fare una review su https:/ /code.launchpad .net/~agilebg/ account- payment/ adding_ account_ due_list_ 7/+merge/ 182330
l 72-127: sono campi/modifiche che verrebbero introdotte da account_due_list, quindi, se lasciamo la dipendenza, non sono necessari
l 207: è forse una modifica che risolverebbe qualche bug conosciuto? La terrei comunque fuori per ora
l 311, 319, 328, 336, 631: come mai viene passato il self alla chiamata ad un metodo?
l 353, 728: anche questa riga risolve forse un bug esistente?
l 440: la scriverei così: creditor_province = creditor_ address. province and creditor_ address. province. code or ''
l 649, 658: è necessario per il porting alla 7?
l 745-750: probabilmente un enhancement da tenere fuori da questa MP
PS: non eliminare questa MP in caso ne volessi fare un'altra, al massimo la impostiamo a rejected. Ti consiglio comunque di fare le modifiche direttamente su lp:~enlightx/openobject-italia/7.0-l10n_it_ricevute_bancarie e questa MP verrà aggiornata automaticamente.
PPS: se hai bisogno di verificare il diff tra versione 6.1 e 7.0, puoi copiare nel branch 6.1 il modulo dal branch 7.0 e fare bzr diff