Code review comment for lp:~vauxoo/addons-vauxoo/7.0-addons-vauxoo-imp-datetime-dev-ernesto
Revision history for this message
| Jorge Angel Naranjo Rogel - http://www.vauxoo.com (jorge-nr) wrote | # |
review:
Needs Fixing
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0e1f616
(Get the code!)
@Wicho
De la linea 51-58 y de la 63-71 estamos repitiendo el código, lo mejor sería crear una función que reciba el datetime y si no viene sea False, y retorne el datetime en base al tz del usuario, este cambio nos puede ayudar a utilizar esa función para otros modelos y no solo para invoice.
Realiza ese ajuste y me informas cuando quede para poder generar los test yaml