Envelope-to: miguel@ssf.com.mx
Delivery-date: Wed, 06 May 2015 23:05:22 -0500
MIME-Version: 1.0
From: Ricardo Sansores <ricardo@ssf.mx>
Date: Wed, 6 May 2015 23:05:00 -0500
Message-ID: <CAKRZ3-PVwU-0uwJA_DvK_NFbE=xih+kGA8A+dypHz-Wwy-vtAQ@mail.gmail.com>
Subject: Comentarios en pull request
To: miguel@ssf.com.mx, =?UTF-8?B?SXNhw61hcyBJw7HDrWd1ZXo=?= <isaiaslee@ssf.mx>
Content-Type: multipart/alternative; boundary=e89a8f64700931dd9105157605b5
 
 Content preview:  Ya están los comentarios en el pull request. Hay que revisarlos
    y corregir. No revisé 2 cosas: 1. Templates y código Javascript: Creo que
    debe hacerlo alguien más ya que no estoy familiarizado con los estándares
    que están usando para comunicación AJAX y validaciones de forma. Recomiendo
    que lo haga alguien de los que está haciendo refactor a todo el proyecto.
    [...] 
 
 Content analysis details:   (-0.7 points, 5.0 required)
 
  pts rule name              description
 ---- ---------------------- --------------------------------------------------
  0.0 HTML_MESSAGE           BODY: HTML included in message
 -0.7 RCVD_IN_DNSWL_LOW      RBL: Sender listed at http://www.dnswl.org/, low
                             trust
                             [209.85.214.170 listed in list.dnswl.org]
 -0.0 RCVD_IN_MSPIKE_H3      RBL: Good reputation (+3)
                             [209.85.214.170 listed in wl.mailspike.net]
 -0.0 RCVD_IN_MSPIKE_WL      Mailspike good senders
X-Spam-Flag: NO

--e89a8f64700931dd9105157605b5
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Ya est=C3=A1n los comentarios en el pull request. Hay que revisarlos y corr=
egir.
No revis=C3=A9 2 cosas:

1. Templates y c=C3=B3digo Javascript: Creo que debe hacerlo alguien m=C3=
=A1s ya que
no estoy familiarizado con los est=C3=A1ndares que est=C3=A1n usando para
comunicaci=C3=B3n AJAX y validaciones de forma. Recomiendo que lo haga algu=
ien
de los que est=C3=A1 haciendo refactor a todo el proyecto.

2. Funcionalidad: El review fue 100% a nivel t=C3=A9cnico y c=C3=B3digo. Ha=
y que
asegurarse que el componente es funcional, c=C3=B3modo de usar, libre de
errores, etc.

Saludos
RSA

--e89a8f64700931dd9105157605b5
Content-Type: text/html; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div><div><div><div>Ya est=C3=A1n los comentarios en el pu=
ll request. Hay que revisarlos y corregir. No revis=C3=A9 2 cosas:<br><br><=
/div>1. Templates y c=C3=B3digo Javascript: Creo que debe hacerlo alguien m=
=C3=A1s ya que no estoy familiarizado con los est=C3=A1ndares que est=C3=A1=
n usando para comunicaci=C3=B3n AJAX y validaciones de forma. Recomiendo qu=
e lo haga alguien de los que est=C3=A1 haciendo refactor a todo el proyecto=
.<br><br></div>2. Funcionalidad: El review fue 100% a nivel t=C3=A9cnico y =
c=C3=B3digo. Hay que asegurarse que el componente es funcional, c=C3=B3modo=
 de usar, libre de errores, etc.<br><br></div>Saludos<br></div>RSA<br></div=
>

--e89a8f64700931dd9105157605b5--