search

fiuba / alfred

GNU General Public License v3.0
3 stars 8 forks source link

Row counter added to students and corrections grid #89

Closed nicopaez closed 10 years ago

anero commented 10 years ago

Está OK, lo único que cambiaría antes de mergear es extraer la función countRows a un JS separado para evitar tener código duplicado. Yo lo puedo hacer esta noche.

nicopaez commented 10 years ago

ok, pero warning porque no es tan directo, vas a tener que ponerle el ID la grilla como parametro.

Saludos! NicoPaez http://nicopaez.wordpress.com/

2014-04-07 9:58 GMT-03:00 Diego Marcet notifications@github.com:

Está OK, lo único que cambiaría antes de mergear es extraer la función countRows a un JS separado para evitar tener código duplicado. Yo lo puedo hacer esta noche.

Reply to this email directly or view it on GitHubhttps://github.com/fiuba/alfred/pull/89#issuecomment-39726278 .

anero commented 10 years ago

Sí lo vi, pero le agregamos una clase a las grillas y usamos eso en el CSS selector.

2014-04-07 10:00 GMT-03:00 Nicolas Paez notifications@github.com:

ok, pero warning porque no es tan directo, vas a tener que ponerle el ID la grilla como parametro.

Saludos! NicoPaez http://nicopaez.wordpress.com/

2014-04-07 9:58 GMT-03:00 Diego Marcet notifications@github.com:

Está OK, lo único que cambiaría antes de mergear es extraer la función countRows a un JS separado para evitar tener código duplicado. Yo lo puedo hacer esta noche.

Reply to this email directly or view it on GitHub< https://github.com/fiuba/alfred/pull/89#issuecomment-39726278> .

— Reply to this email directly or view it on GitHubhttps://github.com/fiuba/alfred/pull/89#issuecomment-39726423 .

diegosanchez commented 10 years ago

El build en travis se rompió. De nuevo el problema con 'solucions'.

Hay algún problema si agrego info de debug para el ambiente ':travis'. Hoy no existe.

2014-04-07 10:01 GMT-03:00 Diego Marcet notifications@github.com:

Sí lo vi, pero le agregamos una clase a las grillas y usamos eso en el CSS selector.

2014-04-07 10:00 GMT-03:00 Nicolas Paez notifications@github.com:

ok, pero warning porque no es tan directo, vas a tener que ponerle el ID la grilla como parametro.

Saludos! NicoPaez http://nicopaez.wordpress.com/

2014-04-07 9:58 GMT-03:00 Diego Marcet notifications@github.com:

Está OK, lo único que cambiaría antes de mergear es extraer la función countRows a un JS separado para evitar tener código duplicado. Yo lo puedo hacer esta noche.

Reply to this email directly or view it on GitHub< https://github.com/fiuba/alfred/pull/89#issuecomment-39726278> .

Reply to this email directly or view it on GitHub< https://github.com/fiuba/alfred/pull/89#issuecomment-39726423> .

Reply to this email directly or view it on GitHubhttps://github.com/fiuba/alfred/pull/89#issuecomment-39726527 .

Saludos DiegoS

[image: Twitter icon] https://twitter.com/dsminotauro [image: LinkedIn icon] http://ar.linkedin.com/in/ingdiegosanchez

blog: http://thoughtprovokingdevelopment.wordpress.com/

anero commented 10 years ago

Por otro lado, me parece que la única razón por la que mantenemos Travis es para buildear los feature branches, el otro día justamente vi que hay un par de opciones para hacer este tipo de cosas con Jenkins así que en vez de tener 2 CI servers distintos podríamos optar por usar solo Jenkins y agregarle esta funcionalidad... Nico, vos que sos el "Jenkins expert", hiciste esto alguna vez?

Saludos, Diego.

2014-04-07 10:23 GMT-03:00 DiegoS notifications@github.com:

El build en travis se rompió. De nuevo el problema con 'solucions'.

Hay algún problema si agrego info de debug para el ambiente ':travis'. Hoy no existe.

2014-04-07 10:01 GMT-03:00 Diego Marcet notifications@github.com:

Sí lo vi, pero le agregamos una clase a las grillas y usamos eso en el CSS selector.

2014-04-07 10:00 GMT-03:00 Nicolas Paez notifications@github.com:

ok, pero warning porque no es tan directo, vas a tener que ponerle el ID la grilla como parametro.

Saludos! NicoPaez http://nicopaez.wordpress.com/

2014-04-07 9:58 GMT-03:00 Diego Marcet notifications@github.com:

Está OK, lo único que cambiaría antes de mergear es extraer la función countRows a un JS separado para evitar tener código duplicado. Yo lo puedo hacer esta noche.

Reply to this email directly or view it on GitHub< https://github.com/fiuba/alfred/pull/89#issuecomment-39726278> .

Reply to this email directly or view it on GitHub< https://github.com/fiuba/alfred/pull/89#issuecomment-39726423> .

Reply to this email directly or view it on GitHub< https://github.com/fiuba/alfred/pull/89#issuecomment-39726527> .

Saludos DiegoS

[image: Twitter icon] https://twitter.com/dsminotauro [image: LinkedIn icon] http://ar.linkedin.com/in/ingdiegosanchez

blog: http://thoughtprovokingdevelopment.wordpress.com/

— Reply to this email directly or view it on GitHubhttps://github.com/fiuba/alfred/pull/89#issuecomment-39728539 .

nicopaez commented 10 years ago

Si, efectivamente se puede hacer con Jenkins. El enfoque es: tener un job que build develop y otro job que buildea todos los demas branches. El job que buildea develop es el que determina el estado del proyecto que se muestra en la página de github. Veo que ajustarlo esta noche.

nicopaez commented 10 years ago

¿en que estado está esto? ¿pudiste verlo Diego? Sino avisame y lo hago yo, quisiera pasar la funcionalidad de contar filas a preview para testearla posta y ponerla en producción lo antes posible, pues la repartija de TPs se hace muy densa sin el contador.

diegosanchez commented 10 years ago

Lo que agregaría/corregiría:

2014-04-13 17:04 GMT-03:00 Nicolas Paez notifications@github.com:

En que estado está esto. Quisiera pasar la funcionalidad de contar filas a preview para testearla posta y ponerla en producción lo antes posible, pues la repartija de TPs se hace muy densa sin el contador.

Reply to this email directly or view it on GitHubhttps://github.com/fiuba/alfred/pull/89#issuecomment-40318073 .

Saludos DiegoS

[image: Twitter icon] https://twitter.com/dsminotauro [image: LinkedIn icon] http://ar.linkedin.com/in/ingdiegosanchez

blog: http://thoughtprovokingdevelopment.wordpress.com/

nicopaez commented 10 years ago

Agree salvo las pruebas de aceptacion pues son muy costosas y esto funcionalidad no lo amerita.

anero commented 10 years ago

Hoy hago el cambio que me habia comprometido en hacer para eliminar la duplicacion de codigo y lo mergeo.

On Sun, Apr 13, 2014 at 6:49 PM, Nicolas Paez notifications@github.comwrote:

Agree salvo las pruebas de aceptacion pues son muy costosas y esto funcionalidad no lo amerita.

— Reply to this email directly or view it on GitHubhttps://github.com/fiuba/alfred/pull/89#issuecomment-40321035 .

nicopaez commented 10 years ago

Tenia un rato libre y hice hace un rato, aunque no hice el merge del branch. ahora estoy luchando porque al correr las pruebas de aceptación del otro branch, el jenkins murio y no logro que levante.

Saludos! NicoPaez http://nicopaez.wordpress.com/

2014-04-13 21:26 GMT-03:00 Diego Marcet notifications@github.com:

Hoy hago el cambio que me habia comprometido en hacer para eliminar la duplicacion de codigo y lo mergeo.

On Sun, Apr 13, 2014 at 6:49 PM, Nicolas Paez notifications@github.comwrote:

Agree salvo las pruebas de aceptacion pues son muy costosas y esto funcionalidad no lo amerita.

Reply to this email directly or view it on GitHub< https://github.com/fiuba/alfred/pull/89#issuecomment-40321035> .

Reply to this email directly or view it on GitHubhttps://github.com/fiuba/alfred/pull/89#issuecomment-40325009 .