search

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

Pre parser #21

Closed colltoaction closed 7 years ago

colltoaction commented 7 years ago

La idea de este PR es hacer un proceso de pre-parseo de los archivos de datos (que no son CSV, es un formato custom), y que en el frontend simplemente se haga request de un JSON.

Ahora organizador.js tiene muchas líneas menos!

Estaría copado seguir un poco este laburo. Por ejemplo, habría que pre-parsear las carreras en lugar de hacer un text search. O usar objetos en lugar de arrays. Etc..

@francoSpada o quien quiera, si pueden chequear que todo siga funcionando sería genial. Yo lo probé y parecía todo bien. Gracias!

colltoaction commented 7 years ago

Le pasé un auto-format al código y el diff quedó inusable. quizás convenga hacer una validación por funcionalidad.

Me parece que el tema de guardar y cargar el estado lo dejé roto.

colltoaction commented 7 years ago

Ya lo arreglé. No es que haya pulido demasiado el código, pero tiene algunas mejoras.

Creo que lo más importante es que tenemos los archivos JSON que tienen un formato sencillo de consumir. Las ventajas son:

  1. Podría reescribirse de 0
  2. Se puede aplicar en otros escenarios, como hacer un heatmap con los horarios más calientes jaja
edittler commented 7 years ago

Concuerdo con las ventajas del formato JSON. Por mi le doy el ok al PR. Podríamos esperar hasta el finde por si alguien más hace alguna observación.

colltoaction commented 7 years ago

Gracias! Ayer intenté extraer el código de las superposiciones pero era imposible jaja