Skip to content

CIT - Cambios en motivo de suspensión de agenda#2243

Open
MarianoCampetella wants to merge 1 commit into
masterfrom
CIT-416
Open

CIT - Cambios en motivo de suspensión de agenda#2243
MarianoCampetella wants to merge 1 commit into
masterfrom
CIT-416

Conversation

@MarianoCampetella

Copy link
Copy Markdown
Contributor

Requerimiento

https://proyectos.andes.gob.ar/browse/CIT-416

Funcionalidad desarrollada

  1. Se modifica esquema de turno para permitir agregar los motivos de forma dinámica.

UserStories llegó a completarse

  • Si
  • No

Requiere actualizaciones en la base de datos

  • Si
  • No

Note

Agregar las siguientes JSONs a la colección "CONSTANTES"
constantes.txt

@nicolasarana

Copy link
Copy Markdown
Contributor
image Funcionando todo ok

@negro89 negro89 left a comment

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.

Campe, la lógica está de 10. Lo que habría que corregir es el esquema de turnos en conjunto con las key de las constantes:

  1. Las keys de cada registro deberían ser representativas del nombre. Si guardamos solo un número y mas adelante el nombre cambia perdemos trazabilidad. Por ejemplo la key de "Gestión de agendas por gestión de pacienes" debería ser algo como 'gestion-de-pacientes' sin mayuscilas ni espacios (aunque no se esté respetando del todo en la colección, es algo que tenemos que corregir).

  2. De la misma manera, el enumerado en motivoSuspencion debería seguir existiendo, pero con las nuevas keys. De esta menera aseguramos que no se guarde basura, pero podemos seguir modificando dinamicamente el texto de los motivos desde el campo nombre si asi lo pidieran.

Cualquier cosulta decime y lo vemos 👍🏼

@negro89 negro89 added the changes requested Se solicitaron cambios label Jun 25, 2026
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.

3 participants