Skip to content

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

Open
nicolasarana wants to merge 3 commits into
masterfrom
RUP-530
Open

RUP - Agregar la posibilidad de cargar plantillas a los hallazgos#3360
nicolasarana wants to merge 3 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

@nicolasarana nicolasarana added changes done and removed changes requested Se solicitaron cambios labels Jun 8, 2026
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.

3 participants