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

Mostrar solamente estudiantes activos en la lista #154

Open
fabianabarca opened this issue Nov 14, 2022 · 2 comments
Open

Mostrar solamente estudiantes activos en la lista #154

fabianabarca opened this issue Nov 14, 2022 · 2 comments
Assignees

Comments

@fabianabarca
Copy link
Owner

Actualmente está mostrando a todas las personas, pero debe ser:

  • A estudiantes solo les muestra estudiantes activos
  • A profesores les muestra primero una lista de estudiantes activos y luego una lista de finalizados y los otros estados.
@EddyRuizB76880 EddyRuizB76880 self-assigned this Jan 3, 2023
@EddyRuizB76880
Copy link
Collaborator

EddyRuizB76880 commented Jan 8, 2023

En la rama eddy_dev están los cambios que resuelven este issue.

@fabianabarca
Copy link
Owner Author

Hola. Algunas observaciones:

  • En estudiantes/views.py la línea 27 hace una primera importación de estudiantes (user__is_staff=False) activos (estado='A'):
estudiantes = Estudiante.objects.all().filter(user__is_staff=False, estado='A')

pero luego vuelve a hacer otras importaciones en los casos de que son profesor o estudiante. Lo mejor sería tener una sola importación de estudiantes (una única vez) y luego hacer la clasificación que existe en cuentas/models.py:

ESTADOS = (
        ('A', 'Activo'),
        ('F', 'Finalizado'),
        ('P', 'Prórroga'),
        ('R', 'Retirado'),
        ('I', 'Incompleto'),
    )

Esta clasificación por estado se puede hacer directamente en el template estudiantes.html con las herramientas de Django para filtrar, o bien filtrar desde views.py.

Noté que está: activos, finalizados y otros estados. Esto está bien, pero mejor si hay una tabla para cada uno, porque las tablas pueden crecer con el tiempo.

En todo caso, está bien la separación hecha hasta ahora, gracias, pero por favor hacer estos ajustes y también ajustes de formato para que sea idéntico (en cuanto a títulos) al documento solicitudes/solicitudes.html.

Nota: favor usar la convención de nombres de aquí entonces sería estudiantes_finalizados, etc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

2 participants