tomgranuja / CayumanDjango

School workshops inscription web form for students enrollment.
0 stars 0 forks source link

added field Period.enrollment_end / modified validation logic in Peri… #39

Closed ralamosm closed 4 months ago

ralamosm commented 4 months ago

@tomgranuja como dije en el issue #35 , si no hay objecion con respecto a agregar campo Period.enrollment_end, pasare esto a produccion.

Para este issue no solo agregue el campo si no que arregle y agregue tests y modifique logica asociada con el manejo de la fin de fecha de inscripcion de Periodo. Hice pruebas manuales ademas de los tests y todo parece funcionar bien.

tomgranuja commented 4 months ago

@tomgranuja como dije en el issue #35 , si no hay objecion con respecto a agregar campo Period.enrollment_end, pasare esto a produccion.

Para este issue no solo agregue el campo si no que arregle y agregue tests y modifique logica asociada con el manejo de la fin de fecha de inscripcion de Periodo. Hice pruebas manuales ademas de los tests y todo parece funcionar bien.

De acuerdo, pasémoslo a producción. Para probarlo localmente (despues de git pull), tendría que correr las migrations primero y después cargar el enrollment_end de cada período?

ralamosm commented 4 months ago

@tomgranuja para probar local tendrias que:

$ poetry run python manage.py shell
......blah blah Python........
>>> import datetime
>>> from cayuman.models import Period
>>> p = Period.objects.all().first()  # o Period.objects.get(id=1), como acomode segun tu DB local
>>> p.enrollment_end = datetime.datetime(2024, 3, 28).date()
>>> p.save()
CTRL+D

Y probablemente debas reiniciar tu django server local y ya. Despues de aplicar el cambio todo deberia funcionar igual que hasta ahora.