Skip to content

RUP - Agregar la posibilidad de cargar plantillas a los hallazgos - #3360

Merged
negro89 merged 4 commits into
masterfrom
RUP-530
Sep 23, 2026
Merged

negro89 merged 4 commits into
masterfrom
RUP-530

Conversation

@nicolasarana

Copy link
Copy Markdown
Contributor

Requerimiento

RUP-530

Funcionalidad desarrollada

  1. Se agregaron los semantig tag agrupados en hallazgos: eventos, situaciones, hallazgos. Para poder cargar plantillas y/o links a ese tipo de conceptos.
  2. Ahora se corrigió para que haga una mejor búsqueda

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

<div class="actions">
<ng-container
*ngIf="procedimiento.semanticTag === 'procedimiento' || procedimiento.semanticTag === 'régimen/tratamiento'">
<ng-container *ngIf="procedimiento.semanticTag === 'procedimiento' || procedimiento.semanticTag === 'régimen/tratamiento' || procedimiento.semanticTag === 'situación' || procedimiento.semanticTag === 'hallazgo' || procedimiento.semanticTag === 'evento'">

@Fabio-Ramirez Fabio-Ramirez May 29, 2026 •

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.

Compa todo ok, solamente el detalle en este caso para descomprimir codigo en la vista y depositarlo en el componente.

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 grcias por la sugerencia


cargarPlantillas(procedimiento) {

cargarPlantillas(procedimiento: any) {

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.

Como pude observar en otros lados que se utiliza la interfaz de Snomed "ISnomedConcept" lo utilizaria aca para reemplazar el any.

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.

Gracias por la observación, listo.

@MarianoCampetella

MarianoCampetella commented Jun 3, 2026 •

Copy link
Copy Markdown
Contributor

La regla de “qué semantic tags soportan plantillas” quedó repetida varias veces.
Acá dejaría la validación centralizada en una constante o helper compartido, porque la misma lista de semantic tags aparece repetida en varios componentes. Si mañana cambia la regla, hay muchas chances de que quede inconsistente entre pantallas. Por ejemplo en un nuevo archivo llamado semantic-tags.helper.ts agregaría lo siguiente:

export const SEMANTIC_TAGS_CON_PLANTILLA = [
'procedimiento',
'elemento de registro',
'régimen/tratamiento',
'situación',
'hallazgo',
'evento'
];

export function permitePlantilla(semanticTag: string): boolean {
return SEMANTIC_TAGS_CON_PLANTILLA.includes(semanticTag);
}

@MarianoCampetella MarianoCampetella added the changes requested Se solicitaron cambios label Jun 3, 2026
@nicolasarana

Copy link
Copy Markdown
Contributor Author
  1. Se agregaron los semantig tag agrupados en hallazgos: eventos, situaciones, hallazgos. Para poder cargar plantillas y/o links a ese tipo de conceptos.

La regla de “qué semantic tags soportan plantillas” quedó repetida varias veces. Acá dejaría la validación centralizada en una constante o helper compartido, porque la misma lista de semantic tags aparece repetida en varios componentes. Si mañana cambia la regla, hay muchas chances de que quede inconsistente entre pantallas. Por ejemplo en un nuevo archivo llamado semantic-tags.helper.ts agregaría lo siguiente:

export const SEMANTIC_TAGS_CON_PLANTILLA = [ 'procedimiento', 'elemento de registro', 'régimen/tratamiento', 'situación', 'hallazgo', 'evento' ];

export function permitePlantilla(semanticTag: string): boolean { return SEMANTIC_TAGS_CON_PLANTILLA.includes(semanticTag); }

Listo Campe :D

@juanGitlein

Copy link
Copy Markdown

Hola Nico, un par de cositas:

  1. Los conceptos que no son procedimientos no deberían permitir cargar una solicitud como plantilla
image
  1. Cargué una plantilla y agregué texto pero al seleccionarla desde el concepto no muestra el "Texto de plantilla"
image
  1. Solo me funcionó correctamente al cargar un enlace externo.

@nicolasarana
nicolasarana force-pushed the RUP-530 branch 3 times, most recently from 42dbec4 to f83ff23 Compare September 17, 2026 12:44
@nicolasarana

Copy link
Copy Markdown
Contributor Author

Hola Nico, un par de cositas:

  1. Los conceptos que no son procedimientos no deberían permitir cargar una solicitud como plantilla
image 2. Cargué una plantilla y agregué texto pero al seleccionarla desde el concepto no muestra el "Texto de plantilla" image 3. Solo me funcionó correctamente al cargar un enlace externo.

Solucionado

@negro89
negro89 merged commit dc8415d into master Sep 23, 2026
1 check passed
@negro89
negro89 deleted the RUP-530 branch September 23, 2026 18:39
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.

5 participants