← Back to team overview

openobject-italia-core-devs team mailing list archive

Re: [Merge] lp:~scigghia/openobject-italia/6.1-openobject-italia-adding-stampa-distinte into lp:openobject-italia/6.1

 

Review: Needs Fixing

Ciao Andrea, grazie del modulo.

In riba_parser.py se non sbaglio newlined_text e page_number sono superflui.

In riepilogo_scadenze_distinte_riba.mako prevederei la traduzione dei termini (a parte il termine 'ricevute bancarie' che lo abbiamo sempre lasciato così, secondo me giustamente), perchè se in un'azienda ci fosse un utente che non parla italiano, non capirebbe la stampa.
Discorso simile per il codice: se dovesse contribuire uno sviluppatore che non sa l'italiano, sarebbe in difficoltà nel leggere il codice.
So che il modulo l10n_it_ricevute_bancarie ha varie stringhe in italiano sparse qua e là, infatti credo dovremmo pulire anche lui prima o poi.
Ho aperto una issue su odoo-italia https://github.com/odoo-italia/odoo-italia/issues/1

C'è la possibilità di avere il modulo anche per la 7?
-- 
https://code.launchpad.net/~scigghia/openobject-italia/6.1-openobject-italia-adding-stampa-distinte/+merge/220517
Your team OpenERP Italia core devs is subscribed to branch lp:openobject-italia/6.1.


References