FdelMazo / FIUBA-Plan

Organizador de horarios de la Facultad de Ingenieria
https://fede.dm/FIUBA-Plan/
MIT License
45 stars 6 forks source link

Allow periodo lectivo selection on horarios SIU #94

Closed FdelMazo closed 5 months ago

FdelMazo commented 5 months ago

Este PR flexibiliza el parser del SIU, ahora fijandose de dividir las materias segun el período lectivo de los horarios. Tuve que reescribir el parser entero, asi que si se puede probar contra otros horarios del SIU que no sean los mios, estaria buenisimo doblechequear que este andando correctamente.

Se puede testear con siu-fede.txt .

La idea es que cuando cargas los horarios, si tenes mas de un periodo lectivo en el SIU, te salte un Select para elegir que periodo usar, y despues eso se use como los horarios del resto de la app.

image

También le agrego el timestamp de cargada y un pequeño warning donde antes decía la fecha de actualización

image

lopezac commented 5 months ago

Hola @FdelMazo, medio raro pero en la branch refactor colocar siu-fede.txt falla como lo esperado, y siu-axel.txt anda bien. Pero en la branch periodos-lectivos no funciona ni siu-axel.txt ni siu-fede.txt, tirando el error de "Error al parsear los horarios del siu" y la warning de "No se encontraron horarios en el archivo".

No te pasa a vos tambien?, o sera mi compu especificamente.

FdelMazo commented 5 months ago

Uh, que bueno que lo probaste @lopezac .

Si queres testear de nuevo, sirve un monton. Gracias!

lopezac commented 5 months ago

Ahora funciona perfecto!