[BUG] View copy_appointments devuelve status code 500

[marcorichetta]

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:

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

https://github.com/cluster311/ggg/blob/2b7bc1884ba2775b8a1e904481e4af2f24cde7cc/calendario/views.py#L108

https://github.com/cluster311/ggg/blob/2b7bc1884ba2775b8a1e904481e4af2f24cde7cc/calendario/views.py#L113

---

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

https://github.com/cluster311/ggg/blob/2b7bc1884ba2775b8a1e904481e4af2f24cde7cc/calendario/views.py#L120

---

• [ ] 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]

@avdata99 Ahí pude hacer funcionar el botón Cargar semana anterior.

• Por el momento copia (sobrescribe) los turnos disponibles y cancelados de la semana anterior a la actual.
• A los turnos de la semana anterior no los toca. Hay un #TODO sobre qué hacer con esos turnos https://github.com/cluster311/ggg/blob/2b7bc1884ba2775b8a1e904481e4af2f24cde7cc/calendario/views.py#L120

Pregunto porque no sé cual sería la funcionalidad correcta.

[avdata99]

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]

@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.