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

Courts organization #24

Closed
wants to merge 25 commits into from
Closed

Courts organization #24

wants to merge 25 commits into from

Conversation

cmglezpdev
Copy link
Contributor

@cmglezpdev cmglezpdev commented Nov 20, 2022

Funcionalidades nuevas

  • Se añadió una sección para poder crear tribunales y editar los ya creados. Estos requieren privilegios de administrador.
  • Por cada tribunal se deben añadir:
    • tesis, miembros del tribunal, fecha, hora, tiempo de duración, local para la discución de la tesis
  • Al crear un tribunal este verifica que:
    • No cree dos tribunales para una misma tesis.
    • Dos personas no esten en dos tesis al mismo tiempo
    • No existan dos tesis en el mismo local al mismo tiempo
  • Se añadió una vista para ver todos los tribunales, estos se pueden organizar por cualquiera de sus propiedades y también ser filtrados por Miembros del tribunal y lugar

Con respecto al código se añadieron dos modelos nuevos:
- Court: representa un tribunal
- Place: representa los sitios donde se discutirán las tesis

Copy link
Member

@apiad apiad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Está espectacular! Lo único que quisiera cambiar es que en vez de impedir guardar un tribunal si hay conflictos, simplemente de un warning en el momento en que se seleccionan fecha / lugar / personas que crearían conflictos pero aún así dejar guardarlo.

Luego en la vista donde están todos los tribunales, poner un ícono al lado de los tribunales que tengan algún conflicto.

@apiad
Copy link
Member

apiad commented Nov 21, 2022

Closes #22

@apiad apiad linked an issue Nov 21, 2022 that may be closed by this pull request
5 tasks
@cmglezpdev
Copy link
Contributor Author

Cambios agregados

  • Se añadió el mensaje de warning al crear la tesis pero aun asi se puede crear(en los caso que se especificaron)
  • Se añadió el ícono de warning al principio del nombre de la tesis en la vista general de todos tribunales
  • También cambié la comparación que hacía entre las fechas para que se pudiera añadir un tribunal justo en el último minuto que termina la anterior o añadir una antes y que termine en el minuto que empiece la siguiente(antes esto decía que estaba ocupada)

@cmglezpdev cmglezpdev closed this by deleting the head repository Jan 20, 2023
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

Successfully merging this pull request may close these issues.

Organización de tribunales de tesis
2 participants