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

MISC - Modificar ficha epidemiológica #3017

Merged
merged 5 commits into from
Aug 8, 2024
Merged

MISC - Modificar ficha epidemiológica #3017

merged 5 commits into from
Aug 8, 2024

Conversation

Fabio-Ramirez
Copy link
Contributor

Requerimiento

https://proyectos.andes.gob.ar/browse/MISC-338

Funcionalidad desarrollada

  1. Se agrega la tabla de contactos estrechos en el componente de ficha generica.
  2. Se corrige la seccion usuario, para que se precarguen datos

UserStory llegó a completarse

  • Si
  • No
  • No corresponde

Requiere actualizaciones en la base de datos

  • Si: En la coleccion formResource modificar usuario --> {
    "_id" : ObjectId("600aca822f344454af894c21"),
    "activo" : true,
    "type" : "section",
    "name" : "Usuario",
    "id" : "usuario",
    "preset" : "usuario",
    "__v" : 0.0
    }
  • No

Requiere actualizaciones en la API

  • Si
  • No

Requiere actualizaciones en andes-test-integracion

  • Si
  • No

if (sec.name !== SECCION_CONTACTOS_ESTRECHOS && sec.name !== SECCION_OPERACIONES) {
const buscado = this.secciones.findIndex(seccion => seccion.name === sec.name);
if (buscado !== -1) {
if (sec.name === SECCION_USUARIO && this.editFicha) {
Copy link
Contributor

Choose a reason for hiding this comment

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

recomiendo usar seccion.id en las comparaciones ya que en varias oportunidades han solicitado cambiar el "label" de la sección

Copy link
Contributor Author

Choose a reason for hiding this comment

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

corregido!

@@ -35,7 +35,7 @@ export class FormPresetResourcesService extends ResourceBaseHttp<Event> {
break;
case 'mpi':
seccion.fields['nacionalidad'] = paciente.direccion[1]?.ubicacion.pais ? paciente.direccion[1].ubicacion.pais : '';
seccion.fields['direccioncaso'] = paciente.direccion[0].valor ? paciente.direccion[0].valor : '';
seccion.fields['direccioncaso'] = paciente.direccion[0]?.valor ? paciente.direccion[0].valor : '';
seccion.fields['lugarresidencia'] = paciente.direccion[0].ubicacion.provincia ? paciente.direccion[0].ubicacion.provincia : '';
seccion.fields['localidadresidencia'] = paciente.direccion[0].ubicacion.localidad ? paciente.direccion[0].ubicacion.localidad : '';
Copy link
Contributor

Choose a reason for hiding this comment

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

en las dos ultimas asignaciones no se esta controlando que exista "paciente.direccion[0]" y esto puede generar un error

Copy link
Contributor Author

Choose a reason for hiding this comment

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

corregido.

@MarianoCampetella
Copy link
Contributor

MarianoCampetella commented Jun 28, 2024

Desde la fecha de "hantavirus" si cargo un nuevo contacto con todos los datos correspondientes y le doy a guardar, si vuelvo a apretar el boton con el icono + me ocurre el siguiente error que con la ficha de covid no me ocurre.
image

Resuelto compa!

@MarianoCampetella MarianoCampetella added changes requested Se solicitaron cambios and removed changes done labels Jun 28, 2024
<plex-table-columns>
</plex-table-columns>
<tr *ngFor="let contacto of contactosEstrechos">
<td *plTableCol="'apellidoNombre'">{{contacto.apellidoNombre}}</td>
Copy link
Contributor

Choose a reason for hiding this comment

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

Usar el pipe de nombre si se puede

Copy link
Contributor

Choose a reason for hiding this comment

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

estos no son datos de pacientes tabulados

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Campe, aca no se podría utlizar el pipe, dado que estos tipos de contactos no cuentan con la info necesaria para un pipe "nombre", es decir que el dato ingresado no cuenta con la esturctucra de nombre, apellido o alias.

</plex-table-columns>
<tr *ngFor="let contacto of contactosEstrechos">
<td *plTableCol="'apellidoNombre'">{{contacto.apellidoNombre}}</td>
<td *plTableCol="'dni'">{{contacto.dni}}</td>
Copy link
Contributor

Choose a reason for hiding this comment

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

Aca se puede utilizar el pipe de documento

Copy link
Contributor

Choose a reason for hiding this comment

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

estos no son datos de pacientes tabulados

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sugerencia aplicada compa, gracias.

@Fabio-Ramirez Fabio-Ramirez added changes done and removed changes requested Se solicitaron cambios labels Jul 2, 2024
Copy link
Contributor Author

@Fabio-Ramirez Fabio-Ramirez left a comment

Choose a reason for hiding this comment

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

TEST Ok!
image
image

@ma7payne
Copy link
Contributor

ma7payne commented Jul 2, 2024

Viendo en conjunto con @MarianoCampetella podemos notar lo siguiente:

  1. Se puede agregar un contacto estrecho sin completar ningún campo, lo que causa un registro con todos valores nulos. Se debería revisar este comportamiento?
  2. Al agregar o eliminar el contecto estrecho, seria conveniente que aparezca un toast con un mensaje? Tal como sucede cuando se modifica la ficha epidemiológica.

Eso es todo! Gracias.

Compas! Se agregan la sugerencias en la corrección.

@MarianoCampetella
Copy link
Contributor

MarianoCampetella commented Jul 12, 2024

Excelente las correcciones Fabi. Solamente te faltó agregar un toast cuando se elimina un contacto estrecho (al momento de seleccionar el boton con el icono del tacho).

Corregido compa!

Copy link
Contributor Author

@Fabio-Ramirez Fabio-Ramirez left a comment

Choose a reason for hiding this comment

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

Copy link
Contributor Author

@Fabio-Ramirez Fabio-Ramirez left a comment

Choose a reason for hiding this comment

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

@negro89
Copy link
Contributor

negro89 commented Aug 7, 2024

Fabi, un par de detallitos que encontré agregando contacto estrecho:

  1. El campo "documento" permite caracteres alfabeticos. Hay un patron en el componente datos-basicos de paciente que te podría servir.
  2. El campo "telefono" permite ingresar solo un cero (por ejemplo). Tambien hay un patron en el componente datos-contacto de paciente que sirve para telefonos.

El resto de 10!

Lauchin! quedaron subidos los cambios.

@negro89 negro89 merged commit cb046aa into master Aug 8, 2024
2 checks passed
@negro89 negro89 deleted the MISC-347 branch August 8, 2024 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Aprobado test ok Los test estan ok
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants