Skip to content

CIT- Visibilidad de agendas de teleconsulta - #2282

Open
nicolasarana wants to merge 2 commits into
masterfrom
CIT-422
Open

CIT- Visibilidad de agendas de teleconsulta#2282
nicolasarana wants to merge 2 commits into
masterfrom
CIT-422

Conversation

@nicolasarana

Copy link
Copy Markdown
Contributor

Requerimiento

Funcionalidad desarrollada

  1. Ahora el profesional puede elegir la opción de mostrar su agenda de telemedicina en mobile o ventanilla de citas

UserStories llegó a completarse

  • Si
  • No

Requiere actualizaciones en la base de datos

  • Si
  • No

Comment thread modules/turnos/routes/agenda.ts Outdated
Comment on lines +249 to +253
if (String(req.query.citasVirtuales) === 'true') {
query.where('bloques.citasVirtuales').equals(true);
} else if (String(req.query.citasVirtuales) === 'false') {
query.where('bloques.citasVirtuales').equals(false);
}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Tal vez este trozo de código se puede simplificar de alguna otra manera como por ejemplo:

const citasVirtuales = String(req.query.citasVirtuales);

if (citasVirtuales === 'true' || citasVirtuales === 'false') {
query.where('bloques.citasVirtuales').equals(citasVirtuales === 'true');
}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ya se corrigió, muchas gracias :D

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.

2 participants