open-source-uc / planner

Un nuevo planner, hecho para estudiantes y por estudiantes.
https://mallas.ing.uc.cl
GNU Affero General Public License v3.0
19 stars 5 forks source link

Mostrar los créditos por semestre #392

Closed kovaxis closed 10 months ago

kovaxis commented 11 months ago

¿Qué se implementa?

Screenshot from 2023-12-06 19-56-26

Opté por no mostrarlo en los semestres en curso por falta de espacio, pero no estoy tan decidido. Mi pensamiento fue que los estudiantes están suficientemente familiarizados con la cantidad de créditos que estan tomando en el semestre actual que vale la pena esconder esta informacion para evitar la tonelada de espacio en blanco que resultaría de agregar los créditos abajo del "En curso".

Screenshot from 2023-12-06 19-56-06

Nota: SIDING no nos da información sobre los cursos actuales, todo esto hay que probarlo con el usuario mock testuser. Cuando SIDING nos de info sobre el semestre actual estos detalles se volveran relevantes.

Prerrequisitos

Esta PR está basada sobre equiv-revamp (porque trabaja sobre código que esta branch también toca, y no quiero trabajar en una version antigua solo para mergearlo a mano despues) y allow-mod-login (porque arregla algunos bugs del manejo de RUTs que no permitían probar los semestres en curso del mock). Supongo que hay que aprobar esas antes.

Update

Implementé el año junto a los créditos, y mostrar (en curso) al lado de los creditos para los semestres en curso:

image

Diegothx commented 10 months ago

Lo único es que se podría colocar "En curso (50 créd)" o "50 créd (En curso)" (?, no sé si opinan que quedaría como muy condensado igual. Fuera de eso, todo bien, apruebo apenas los prerrequisitos estén ready