Simplificar el final de carrera (Tesis / TPP) carreras del plan 2020

FdelMazo / FIUBA-Map

Mapa de materias y sus correlativas de la Facultad de Ingeniería

https://fede.dm/FIUBA-Map/

MIT License

129 stars 37 forks source link

Simplificar el final de carrera (Tesis / TPP) carreras del plan 2020 #210

Closed lopezac closed 5 months ago

lopezac commented 5 months ago

closes #209

Hola @FdelMazo como andas.

Remplace en los respectivos archivos json (en la carpeta data) las materias de TESIS y TPP por TIF
Remplace en carreras.js el finDeCarrera a contener solamente la materia TIF
Movi en carreras.js sistemasviejo al comienzo antes de sistemas (para mantener el orden, viste por ej: esta quimica y abajo quimica-2020)

Encontras algun error o algo que mejorar?

Tengo una duda:

En las carreras del plan 2020 (excepto petroleo-2020) tienen solamente el TIF, por ende seria logico que no haya que elegir el final de carrera (que en el userMenu aparezca solamente Cerrar Sesion), como por ejemplo analista de sistemas 2014, que por defecto esta seleccionado la materia de fin de carrera, Trabajo Profesional.

FdelMazo commented 5 months ago

@lopezac me ganaste de mano, lo iba a implementar mañana yo! lo veo barbaro. El truquito para que no haya que elegir final de carrera cuando hay uno solo es cambiarle la categoria al nodo, a que sea Fin de Carrera (Obligatorio). Si te fijas, en el código en muchos lados se filtra por esa condición.

Lo que queda es cambiar todos los TIFs a esa categoria. Un ejemplo es sistemas-2014, que solo tiene tpp:

https://github.com/FdelMazo/FIUBA-Map/blob/a615e462a0cb39c96ea3a5b284ef53f50d6eee7f/src/data/sistemas-2014.json#L426-L432

También, un cambio que podemos hacer es matar lo de TIF1 y TIF2 de petroleo y dejarlo todo en el mismo. Por mas que el PDF lo aclare como dos materias distintas, creo que esta bien que el fiuba map no haga esa distincion

lopezac commented 5 months ago

@FdelMazo sabes que me preguntaba porque la diferencia entre el Fin de Carrera (Obligatorio) y el Fin de Carrera y intente implementar el detalle que te dije de duda, pero no funcionaba como sistemas-2014, con razon jajaj, mañana lo implemento.

Esta documentado en alguna parte este detalle? o lo pase de alto (seguro)?

Y lo de petroleo tenes razon, es mas simple que sea una sola materia el TIF de pétroleo-2020 para FIUBA-map.

FdelMazo commented 5 months ago

No te pasaste nada por alto! es culpa mía. La verdad que la documentación es floja. Me esforcé en documentar lo que hacen las funciones por separado, pero termino siendo mas ayuda memoria para mi mismo. Nunca agregue un apartado general de los conceptos core de la app.

lopezac commented 5 months ago

Hola @FdelMazo, aca lo que hice fue:

Arregle un typo en petroleo-2020.json que la materia de TIF tenia de correlativa a Probabilidad y Estadistica como "PYE" en vez de "PROBA"
Unifique las 2 partes del TIF de petroleo en una sola materia
Puse todas las carreras del plan-2020 sin finalDeCarrrera, implementado el metodo que se utiliza en sistemas-2014 de poner el TIF en la parte de creditos.materias logrando asi que este seleccionada por defecto.

Se podria abrir un issue para agregar la documentacion core de la app no?