On 12/01/14 06:06, Albert Cervera i Areny wrote:
2014/1/12 oscar_alvarez <[email protected]>:
Hola Devs
Desde hace tiempo vengo haciendo aportes a tryton y todos los modulos a
traves de la traduccion es_CO y no he tenido problemas, pero con la
exportacion de parches al trunk para hacer contribuciones (no traducciones)
he tenido algunos tropiezos, aunque he tratado de hacerlo siguiendo la guia
en la wiki [1] al pie de la letra he obtenido este mensaje de Cedk:
This is not what we call submitting a patch, see
https://code.google.com/p/tryton/wiki/HowtoContribute
eso fue respecto a un patch que intente enviar hace ya unos meses con nombre
de archivo: issue_864002.patch (usando el commando export de mercurial), el
archivo quedo así:
# HG changeset patch
# User Oscar Alvarez <[email protected]>
# Date 1368324877 18000
# Node ID 21c0fc7f22792d89e7b7589136f99a7bd4c756b1
# Parent d3c06443a7b696aff77ec38c3e3fcbcf87ea5192
Set correct invoice_type on invoice line
diff -r d3c06443a7b6 -r 21c0fc7f2279 sale.py
--- a/sale.py Mon Apr 22 12:14:19 2013 +0200
+++ b/sale.py Sat May 11 21:14:37 2013 -0500
@@ -1249,6 +1249,7 @@
invoice_line.product = self.product
invoice_line.unit_price = self.unit_price
invoice_line.taxes = self.taxes
+ invoice_line.invoice_type = invoice_type
if self.product:
invoice_line.account = self.product.acco
unt_revenue_used
if not invoice_line.account:
exporting patch:
<fdopen>
Aunque fue aceptado el codereview [2] con un LGTM, fue rechazado el archivo
del patch por la forma en que lo envie o por como quedo el archivo, no
entiendo realmente cual fue la causa del rechazo si aparentemente segui la
guia de la wiki y esta guia:
http://mercurial.selenic.com/wiki/SpanishTutorialExport
[1]
http://code.google.com/p/tryton/wiki/HowtoContribute#How_to_submit_your_patches/contributions
[2] http://codereview.tryton.org/864002/
Me gustaria saber ya que algunos de ustedes tienen mas experiencia, cual fue
el error y como deberia entonces ser enviado, debido a este inconveniente
estoy limitado en los aportes que puedo hacer, recuerden que yo soy novato
en el tema de revisión de código.
No añadistes el codereview en el mensaje del commit y si había
incidencia bugs.tryton.org tampoco estaba.
Gracias Albert ya entendi, pase por alto ese detalle.
