lugfi / organizador-fiuba

El organizador te ayuda a organizar las materias con un calendario semanal, mostrando superposiciones y permitiendo probar distintas combinaciones de cursos rapidamente.
https://lugfi.github.io/organizador-fiuba/
GNU General Public License v3.0
15 stars 16 forks source link

Actualizo a 2Q2017 en la app y README #31

Closed edittler closed 7 years ago

edittler commented 7 years ago

Además de agregar el json de 2Q2017 en el submenú, tuve que subir la posición del mismo porque la última opción no se mostraba completa.

Resolves #30

fdnk commented 7 years ago

Arreglé un pequeño typo y realicé un comentario en una variable posiblemente en desuso.

colltoaction commented 7 years ago

Si dejan de entrar los botones, no me parece mal sacar los más viejos. Si no, LGTM

edittler commented 7 years ago

¡Gracias @fdnk! Yo acabo de corregir otro typo y creo que ya estamos.

@tinchou podríamos crear un issue donde discutamos qué hacer con los datos de cuatrimestres anteriores así sacamos este PR rápido y habilitamos los nuevos datos en el organizador.

fdnk commented 7 years ago

Diría que subamos esto rápido así ya lo pueden usar, la gente se desespera

El 3/8/2017 8:37, "Ezequiel Dittler" notifications@github.com escribió:

¡Gracias @fdnk https://github.com/fdnk! Yo acabo de corregir otro typo y creo que ya estamos.

@tinchou https://github.com/tinchou podríamos crear un issue donde discutamos qué hacer con los datos de cuatrimestres anteriores así sacamos este PR rápido y habilitamos los nuevos datos en el organizador.

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/lugfi/organizador-fiuba/pull/31#issuecomment-319944598, or mute the thread https://github.com/notifications/unsubscribe-auth/AGsHArq176UyNJ6VQ_x7nxBuzttPSyslks5sUbD2gaJpZM4Or6RU .

colltoaction commented 7 years ago

Gracias! Calculo que no vale la pena preocuparse ahora por ese detalle. Si este cuatrimestre actualizamos la UI seguro que lo corregimos.