IMP certificado de subcontratista

[angelmoya]

Modulo para la gestión de certificado de subcontratista de proveedores. Se ha comentado en:
https://groups.google.com/forum/#!msg/openerp-spain/MNwt9_bAbzk/EGZpkPCreqwJ

[pedrobaeza]

Elimina la marca del editor vim y deja una línea en blanco al final.

[pedrobaeza]

Revisa además los errores PEP8 que indica Travis.

[angelmoya]

Ok, parece que ya está todo, lo que no tengo claro es porque falla travis.

[pedrobaeza]

Travis está fallando porque hay un error en el core con la detección de la librería wkhtmltopdf. Hay issue levantado ya. Lo que estoy viendo es que tal como está el módulo, no es algo específico de la localización, ya que en realidad es un mecanismo de aviso, y podría quitarse el l10n_es del nombre. Además, yo diría que un lugar más apropiado es el repositorio purchase-workflow. ¿Qué opinas?

[angelmoya]

Pues he estado dándole vueltas... y he recordado porque lo había hecho con dos campos, puede darse el caso que den de alta un proveedor, que quieran avisar que no tiene certificado de subcontratista, pero que realmente todavía no lo tiene, por eso primero se marca que es requerido, y cuando se tiene se indica la fecha... voy a darle una vuelta para volver a poner los dos campos, y mejor mostrar dos errores distintos.

Por otro lado, también estuve pensando si meterlo en la localización o en purchase-workflow, realmente en purchase workflow sería mas genérico y creo que podría venir bien, pero si es sólo por añadir una fecha quizás sería mejor una mejora en el módulo warning para mostrar avisos segun la fecha.

Al final pienso que aunque hubiese un módulo genérico para avisos de proveedores en base a una fecha, es útil que sea especifico para esta funcionalidad que si es de la localización española, como comentaba en el hilo está descrito aqui
http://www.agenciatributaria.es/AEAT.internet/Inicio_es_ES/_Configuracion_/_top_/Ayuda/Certificados_Tributarios/Guia_de_usuario__Certificacion_para_Contratistas_y_Subcontratistas/Guia_de_usuario__Certificacion_para_Contratistas_y_Subcontratistas.shtml
y es muy util para empresa que subcontraten su actividad principal (transportes, proyectos de ingenieria, contruccion ...). Teniendo un módulo específico el usuario podría diferenciar si los avisos son por otro motivo o por el certificado de subcontratista.

Como también es módulo muy simplón no creo que requiera mucho mantenimiento y ofrece un mecanismo especifico para esa necesidad.

[pedrobaeza]

Vale, pues distingue el mensaje de error entonces. Y lo de ser l10n_es... igualmente, es cuestión de preferencias. Tú que lo propones, eliges.

[angelmoya]

Ok, ya esta todo

[coveralls]

Coverage increased (+0.4%) when pulling 18df7a9 on AngelMoya-Domatix:subcontractor_certificate into 6d846b4 on OCA:8.0.

[pedrobaeza]

Puedes poner versión 1.0

[angelmoya]

Gracias Pedro, con esto creo que ya no falta nada.

[pedrobaeza]

Gracias!

👍

[coveralls]

Coverage increased (+0.4%) when pulling 01182cb on AngelMoya-Domatix:subcontractor_certificate into 6d846b4 on OCA:8.0.

[rlizana]

Probado

👍