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

[BUG] View copy_appointments devuelve status code 500 #208

Open
4 tasks
marcorichetta opened this issue May 14, 2020 · 3 comments
Open
4 tasks

[BUG] View copy_appointments devuelve status code 500 #208

marcorichetta opened this issue May 14, 2020 · 3 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@marcorichetta
Copy link
Collaborator

marcorichetta commented May 14, 2020

Descripcion del bug
En la vista de /turnos al hacer click en Cargar semana anterior se muestra una notificación con un error de request fallida con status 500

URL del error: http://localhost:8000/turnos/appointments/copy/?start=2020-05-11T00:00:00.000Z&end=2020-05-18T00:00:00.000Z
Screenshot:
imagen

Como reproducir el error

  1. Ingresar a vista de calendario
  2. Apretar botón Cargar semana anterior

Solucion esperada

  • Debe cargar turnos de la semana anterior

Tareas para lograr la solucion

  • Pasarle el parámetro servicio a la función get_appointments_list

current_appointments = get_appointments_list(

appointments = get_appointments_list(


  • Definir que hacer con los turnos de la semana anterior. Actualmente no son modificados.

# TODO: Define what to do with existent appointments


  • Definir que hacer en el caso de haber conflictos entre turnos en el mismo horario.
  • Por el momento copia (sobrescribe) los turnos disponibles y cancelados de la semana anterior a la actual.

Info extra
A través de Ajax se llama a la función copy_appointments, que a su vez llama a la función get_appointments_list y acá se produce el error

TypeError at /turnos/appointments/copy/
get_appointments_list() missing 1 required positional argument: 'servicio'
@marcorichetta marcorichetta added the bug Something isn't working label May 14, 2020
@marcorichetta marcorichetta self-assigned this May 14, 2020
@marcorichetta marcorichetta added this to the v 0.3 milestone May 14, 2020
@marcorichetta

This comment has been minimized.

@avdata99
Copy link
Member

No se si se va a usar mucho esta function @marcorichetta
Entiendo que los anteriores quedan como estaban, no deberían perderse
Para copiarlos a los nuevos habría que validar que los servicios y los profesionales todavía estén disponibles (yo no lo haría ahora, es un lío)
Creo que este issue no es importante, deja marado el error que tenes con el mejor detalle en la descripción de lissue y se verá a futuro

@marcorichetta
Copy link
Collaborator Author

@avdata99 Ahí describí lo mejor que pude la solución al bug y que otras cosas deberían tenerse en cuenta respecto a qué hacer con los turnos existentes.

@avdata99 avdata99 modified the milestones: v 0.3, v 0.4 May 26, 2020
@ramosfm ramosfm self-assigned this Sep 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants