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

LAB - Paciente temporal enviar DNI parentezco #3093

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

JuanIRamirez
Copy link
Contributor

Requerimiento

https://proyectos.andes.gob.ar/browse/LAB-13

Funcionalidad desarrollada

  1. Si el paciente es temporal (sin DNI) envía documento del parentezco (progenitor/a).
  2. SI no hay documento no hace el GET.

UserStory llegó a completarse

  • Si
  • No
  • No corresponde

Requiere actualizaciones en la base de datos

  • Si
  • No

Requiere actualizaciones en la API

  • Si
  • No

Requiere actualizaciones en andes-test-integracion

  • Si
  • No

documento) utiliza documento del prentezco
(progenitor/a)
Copy link
Contributor

@negro89 negro89 left a comment

Choose a reason for hiding this comment

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

Juancho, no probé si andaba porque hay una comparación que no iba a salir bien (linea 456). Dejo un par de observaciones/sugerencias

if (!dni) {
for (const relaciones of this.paciente.relaciones) {
if (relaciones.nombre = 'progenitor/a') {
const pacienteRel = await lastValueFrom(this.pacienteService.getById(relaciones.referencia));
Copy link
Contributor

Choose a reason for hiding this comment

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

Juancho, tenemos un par de temitas por acá ..

  1. No usamos async/await del lado del cliente ya que este comportamiento lo incorporan los observables cuando uno se subscribe.
  2. En las relaciones del paciente viene incluído el documento y los demás datos basicos, por lo que no es estrictamente necesario traer los datos de la DB. En este punto quizas haya que tener en cuenta el numeroIdentificacion para extranjeros que aún no tengan su dni argentino. También posiblemente sea correcto tener en cuenta la relación 'tutor' en caso de que la persona a cargo no sea un progenitor/a.

Dejo sugerencia de codigo:

Suggested change
const pacienteRel = await lastValueFrom(this.pacienteService.getById(relaciones.referencia));
buscarCDAPacientes(token) {
const { estado, fechaNacimiento, apellido } = this.paciente;
const fecNac = moment(fechaNacimiento).format('yyyyMMDD');
const fechaHta = moment().format('yyyyMMDD');
let dni = this.paciente.documento;
if (!dni) {
const tutorProgenitor = this.paciente.relaciones.find(rel => rel.relacion === 'progenitor/a') || this.paciente.relaciones.find(rel => rel.relacion === 'tutor');
// en esta linea, si se quisiera, debería traerse los datos de tutorProgenitor
dni = tutorProgenitor?.documento || tutorProgenitor?.numeroIdentificacion || null;
if (!dni) {
return;
}
}
forkJoin ....

@negro89 negro89 added the changes requested Se solicitaron cambios label Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes requested Se solicitaron cambios
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants