Skip to content

V3.4.0#388

Merged
condorpiedra merged 38 commits intomasterfrom
v3.4.0
Aug 6, 2018
Merged

V3.4.0#388
condorpiedra merged 38 commits intomasterfrom
v3.4.0

Conversation

@condorpiedra
Copy link
Copy Markdown
Contributor

  • Los cambios más relevantes se encuentran en el módulo de citas (punto de inicio de turnos). Se reorganiza la forma de mostrar la información de turnos y carpetas de un paciente.

Funcionalidades ingresadas

Requiere actualizaciones en la base de datos

  • Si
  • No
  • Verificar los permisos para cambiar el número de carpeta en un efector.

NicoDino and others added 30 commits May 28, 2018 13:51
endpoint get historial turnos paciente
mobile: Handler error en login no profesional.
se modifica job de Pecas Consolidado
Comment thread auth/permisos.ts
{ key: 'solicitud', title: 'Registrar solicitud', type: 'boolean' },
{ key: 'autocitado', title: 'Registrar Autocitación', type: 'boolean' },
{ key: 'darTurnos', title: 'Dar turnos desde punto inicio', type: 'boolean' },
{ key: 'puedeEditarCarpeta', title: 'Editar número de carpeta', type: 'boolean' }
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.

La palabra 'puede' en el nombre de un permiso es redundante.

}

];
let data2 = await agenda.aggregate(pipelineTurno).exec();
Copy link
Copy Markdown
Contributor

@liquid36 liquid36 Aug 6, 2018

Choose a reason for hiding this comment

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

Mejora: Se puede hacer una sola query de mongo para obtener el listado de turnos y sobreturnos.

let data2 = await agenda.aggregate(pipelineTurno).exec();
let sobreturnos = await agenda.aggregate(pipelineSobreturno).exec();
data2 = data2.concat(sobreturnos);
data2.forEach(elem => {
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.

Mejora: para evitar este for se podría usar el $project de mongo.

Comment thread package.json
{
"name": "api",
"version": "3.3.2",
"version": "3.4.0",
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.

Hay que jugar el 34 en la lotería. XD

Copy link
Copy Markdown
Contributor

@liquid36 liquid36 left a comment

Choose a reason for hiding this comment

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

Todo bien!
Adjunto algunas sugerencias.

Copy link
Copy Markdown
Contributor

@silviroa silviroa left a comment

Choose a reason for hiding this comment

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

funcionalidas ok!

@condorpiedra condorpiedra merged commit 038e44e into master Aug 6, 2018
@liquid36 liquid36 deleted the v3.4.0 branch June 3, 2019 17:14
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.

5 participants