Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Calculadora de notas #17

Merged
merged 27 commits into from
Mar 27, 2023
Merged

Calculadora de notas #17

merged 27 commits into from
Mar 27, 2023

Conversation

mapacheverdugo
Copy link
Member

@mapacheverdugo mapacheverdugo commented Jan 27, 2023

Descripción

  • Calculadora de notas para las asignaturas

Tipo de cambio

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

Checklist:

  • 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).

@mapacheverdugo mapacheverdugo added the 🆕 New feature New feature or request label Jan 27, 2023
@mapacheverdugo mapacheverdugo self-assigned this Jan 27, 2023
@mapacheverdugo mapacheverdugo linked an issue Jan 27, 2023 that may be closed by this pull request
@mapacheverdugo mapacheverdugo changed the title Feat/calculator Calculadora de notas Jan 27, 2023
@RafaelMoralesV
Copy link
Member

pudiste buildearlo sin problemas?

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

image

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
Copy link
Member

image

nvm, si es un error por actualizar flutter

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

@RafaelMoralesV
Copy link
Member

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
Copy link
Member

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
Copy link
Member Author

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
Copy link
Member Author

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

@RafaelMoralesV
Copy link
Member

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

@mapacheverdugo mapacheverdugo merged commit c6903f6 into dev Mar 27, 2023
@RafaelMoralesV RafaelMoralesV deleted the feat/calculator branch April 6, 2023 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🆕 New feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Calculadora de notas
2 participants