Skip to content

Conversation

@negro89
Copy link
Contributor

@negro89 negro89 commented Dec 2, 2025

Requerimiento

https://proyectos.andes.gob.ar/browse/MISC-374
https://proyectos.andes.gob.ar/browse/PLEX-367
Actualizar build de Plex para ser compilado por ng-packagr (Necesario para compatibilidad con ANDES)

Funcionalidad desarrollada

  1. Migración de angular 12 a 15 y actualizaciones requeridas del package
  2. Instalación de Plex v9.0.0-beta.1
  3. Se excluye proyecto del portal del paciente de la compilación y la integración continua de github

Requiere actualizaciones en la base de datos

  • Si: Antes de deployar, reemplazar color de menú "epidemiología" por #a3991c
  • No

Requiere actualizaciones en la API

  • Si
  • No

OBSERVACION: No es dependencia, pero se puede probar con la actualización a node 18 👉🏼 andes/api#2163

Pasos para la instalación

Primero desde master ...

  1. rm -rf node_modules package-lock.json (borra node_modules y package-lock)
  2. npm cache clean --force (limpia caché)

Ahora nos movemos a angular-v15 y ...

  1. nvm use 18 (usa la version 18 de node) <---- solo app!
  2. npm ci
  3. npm run start:dev (corremos como siempre)

@negro89 negro89 requested review from a team as code owners December 2, 2025 18:28
@negro89 negro89 mentioned this pull request Dec 2, 2025
9 tasks
Copy link
Contributor

@AgosLizzi AgosLizzi left a comment

Choose a reason for hiding this comment

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

Lauchi, van algunos cambios que vimos:

  1. Como primero, te adjunto un videito con del error en consola al ingresar e intentar registrar en rup/huds. Descarté que en demo no este sucediendo ese mismo error. También con diferentes pacientes, se repetía. https://www.loom.com/share/73d18f4c481042619977dd659a60049a
  2. Fíjate que los tabs tienen un pequeño error en cuanto al tamaño; al seleccionarlos, se visualiza mejor (pareciera que el color queda a medias) no importa el módulo, siempre los muestra asi:
tab int tabs
  1. En el componente DateTime, si abrimos y nos queremos quedar con la fecha del día por defecto no aparece seleccionada, pero la imagen del calendario da la sensación que si. Tal vez debería aparecer seleccionada con el celeste directamente y evitar ese pequeño error.
fecha 4. En el gestor de agendas, el texto del hint no respeta el tamaño del componente y queda casi invisible. hint
  1. El help del gestor que muestra la configuración del bloque esta muy grande, se debería ajustar el tamaño. Esto lo vimos juntos
help 6. No estoy segura si este punto corresponde a esta corrección del Pr pero lo detallo por las dudas para que quede y no nos olvidemos. En el Sidebar de búsquedas recientes, ventanilla se solapan los badge, más que nada cuando tenemos el de fallecido (en demo actualmente se comporta de esta manera :( ) badge
  1. Sobre el help del mapa de camas, al editar la fecha la línea del plex title esta a la mitad
help  mapa de camas
  1. El tooltip de "anular inicio de prestación" queda activo una vez que el modal cierra https://www.loom.com/share/3d70327aa4f940febc60e2bae04b86c2

  2. Esto ya lo había mencionado como corrección en el PR MISC-410. Fondo amarillo sobre tipografía blanca es ilegible. Deberíamos optar por un amarillo con otra tonalidad.

amarillo en el menú

@negro89
Copy link
Contributor Author

negro89 commented Feb 10, 2026

@AgosLizzi gracias por la revisión. Ya están las correcciones 👌🏼 algunas con observaciónes 👇🏼 :

  1. La parte que encuadra la fecha está muy enlatada en la librería y no la pude cambiar. Para subsanar el hecho de que a veces la fecha viene pre-seteada y otras veces no (esto es intencional segun cada caso en la app) se deshabilita el botón "aceptar" hasta que se seleccione la fecha (en los casos que no venga por defecto seleccionada).
  2. No era consecuencia de los cambios pero aproveché de corregirlo ✅
  3. ❌ Es algo que venimos arrastrando actualmente. Hay que analizar qué pasa y armar una tareita aparte 👍🏼
  4. ✅ Esto se configura por base de datos. Le modifiqué el color en demo, si no lo ves avisame y lo replicamos.

Cualquier cosita consultame 👌🏼

Copy link
Contributor

@AgosLizzi AgosLizzi left a comment

Choose a reason for hiding this comment

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

Laucha. Los cambios anteriores estan super! Vi todo ok. Te dejo un error en consola que aparecio al querer agregar un número de carpeta y cancelar esa edición *Solo sucede cuando cancelas, al aceptar no hay problema. De yapa, el botón de REGISTRAR NUEVA CARPETA queda muy pegado al título, se podría modificar a REGISTRAR NUEVA? https://www.loom.com/share/f8acb27ca39c4a85b703e88e09abd07d

*El color del punto 9 lo veo igual, pero confio en que esta modificado 🤞

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants