FdelMazo / FIUBA-Plan

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

Materia Arq de Software faltante #91

Closed Naza26 closed 4 months ago

Naza26 commented 4 months ago

Hola!

No sé si me pasa solo a mi pero cuando quise armar un plan, busqué la materia Arquitectura del Software, que entiendo es una optativa tanto en el plan viejo como en el nuevo de la carrera de Ing.Informática, no la encontré. Si mal no recuerdo, el código de materia es 75.73. Me cloné el repo y lo estuve debuggeando un cachito para ver que pasaba. Al principio pensé que faltaba del json de carreras pero después vi que el código de materia esta ahí efectivamente. Asi que vi que cuando crea el dataContext que después consume el buscador, inicializa el source con horariosSiu ó jsonData (horarios), ahi mismo puse un log y vi que no viene la materia directamente (source.materias.filter(materia => materia.codigo == "7573")). Intenté agregarla al json de horarios pensando que eso lo arreglaría pero no vi nada diferente. No sé si tengo que actualizar algo o estoy viendo en el lado correcto. Tampoco me queda claro que esto sea un problema para todos o que quizás sea un tema mío, levanto el issue por las dudas!

FdelMazo commented 4 months ago

Hola Nazaaa, gracias por el issue! El tema de arqui lo tengo que investigar a ver que estuvo fallando, pero mas alla de eso la idea es que en el futuro se saque del todo los horarios cuatrimestrales de la app y solo se tengan los del SIU cargados a manos (se puede ver en el PR abierto). Esto es porque los horarios de FIUBA son cada vez mas dificiles de conseguir y es mejor idea dejar de depender de servicios externos.

Despues de esos cambios y refactor me fijo si hay algun error relacionado a este issue que siga estando. Gracias por el issue!

Naza26 commented 4 months ago

De nada Fede!

Perdón, quise ver de arreglarte el problema antes de cargarte el issue pero no pude. Cualquier cosa que necesites ando por acá!

Beso

FdelMazo commented 4 months ago

Viendo el servicio de donde salen los horarios de FIUBA, parece que arqui no tiene horarios ahí. Como es una materia sin clases, directamente no se agrega al FIUBA-Plan cuando se hace el parseo, para evitar que haya materias fantasma dando vueltas.

image

Pero bueno, la idea es sacar del todo el uso de esa pagina. (Siempre bienvenida la colaboracion al PR abierto!)

Naza26 commented 4 months ago

Genial Fede, si tengo un rato lo chusmeo bien!