exdevutem / mi-utem

Aplicación móvil Mi UTEM de la Universidad Tecnológica Metropolitana de Chile

14 stars 6 forks source link

Calculadora de notas #17

Closed mapacheverdugo closed 1 year ago

mapacheverdugo commented 1 year ago

Descripción

Calculadora de notas para las asignaturas

Tipo de cambio

[ ] Corrección de error.
[x] Nueva funcionalidad .
[ ] Breaking change (cambio que puede romper la aplicación).
[ ] Documentación.
[x] Mejora de codigo existente (performance y/o estilo del codigo y/o otra mejora)

Checklist:

[x] He practicado una revisión de mi propio codigo.
[ ] He comentado mi codigo, sobretodo en las partes que pueden resultar dificiles de entender.
[ ] Mis cambios no generan nuevas alertas (Warnings).
[ ] Este pull request contiene <1000 lineas de codigo (LOC).

RafaelMoralesV commented 1 year ago

pudiste buildearlo sin problemas?

Me tira un error en lib/widgets/custom_drawer.dart

Quiero asumir que es por que actualicé mi versión de flutter? que versión estás corriendo tú? yo estoy en la 3.7.0

RafaelMoralesV commented 1 year ago

nvm, si es un error por actualizar flutter

Voy a pushearte un cambio en ese import al respecto, y poco más

RafaelMoralesV commented 1 year ago

nvm again, supongo que es mejor armar una branch que arregle eso, junto con los deprec nuevos como te amo flutter

voy a intentar tenerla subida esta tarde

RafaelMoralesV commented 1 year ago

Por fin pude probarlo Funciona bacán, aunque noté un detalle!

Puedes agregar notas que sumen más del 100% kdjfngdkf Lo mismo si suman menos del 100%

Me gustaría ver algo tipo << 👀 Uh, revisa; tus notas suman más del 100% / tus notas no suman 100% >> en una especie de tarjeta chiquita que aparece entre la tarjeta de nota de presentación y la tarjeta de lista de notas

Te gustaría que la agregue yo? o te la encargo a ti?

mapacheverdugo commented 1 year ago

Te gustaría que la agregue yo? o te la encargo a ti?

Dale no más, que lo haga el que pueda pushearlo primero, mis tiempos y ganas son muy variables. A ese mensaje también le agregaría cuando una nota no es válida (mayor a 7 por ejemplo)

mapacheverdugo commented 1 year ago

Ya, creé la tarea/issue #20 para los mensajes, así mergeamos esta wea luego xd

RafaelMoralesV commented 1 year ago

Ya, creé la tarea/issue #20 para los mensajes, así mergeamos esta wea luego xd

AKJSKAJ ya voy a penquear al panita que le encargué ese issue, dejo el sipo apruebo por mientras