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

RUP - Bloquea edicion de recetas #3165

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

RUP - Bloquea edicion de recetas #3165

wants to merge 1 commit into from

Conversation

ma7payne
Copy link
Contributor

@ma7payne ma7payne commented Mar 25, 2025

Requerimiento

https://proyectos.andes.gob.ar/browse/RUP-424

Funcionalidad desarrollada

  1. Se bloquea la edición de la receta chequeando si la validación se rompió en la prestacion.
  2. Se crean funciones para validar los conceptos al momento de agregarlos a la prestacion verificando si estan bloqueados para editar/agregar.
  3. Se agrega posibilidad de mostrar un mensaje de alerta en componente RUP

UserStory llegó a completarse

  • Si
  • No
  • No corresponde

Requiere actualizaciones en la base de datos

  • Si
  • No

Requiere actualizaciones en la API

Requiere actualizaciones en andes-test-integracion

  • Si
  • No

@ma7payne ma7payne requested review from a team as code owners March 25, 2025 18:58
@ma7payne ma7payne requested review from MCele and aldoEMatamala March 25, 2025 18:58
@ma7payne ma7payne force-pushed the RUP-424 branch 2 times, most recently from cb0625e to 40972ce Compare March 27, 2025 18:59
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.

Dejo una sugerencia miníma de mejora para el mensaje:

El borde del rectangulo lo veo muy delgado y tal vez no presente la visibilidad que estamos buscando. Probé pasarlo a 3px y la tipografia la pasé a bold, adjunto imagen de muestra. Fijate si te parece bien.
image

Un detalle, no pude acomodar el texto y me quedo una "y" viuda. Si se puede hacer un salto de linea y que termine "y registrar una nueva" en una sola mejor

@ma7payne ma7payne force-pushed the RUP-424 branch 2 times, most recently from 5d53202 to 1bfbb30 Compare April 1, 2025 11:59
@ma7payne
Copy link
Contributor Author

ma7payne commented Apr 1, 2025

Dejo una sugerencia miníma de mejora para el mensaje:

El borde del rectangulo lo veo muy delgado y tal vez no presente la visibilidad que estamos buscando. Probé pasarlo a 3px y la tipografia la pasé a bold, adjunto imagen de muestra. Fijate si te parece bien. image

Un detalle, no pude acomodar el texto y me quedo una "y" viuda. Si se puede hacer un salto de linea y que termine "y registrar una nueva" en una sola mejor

@AgosLizzi ya estan aplicados los cambios en la alerta. El salto de linea no lo incluyo porque rompe la configuracion del texto cuando actua el responsive en resoluciones medias. En la imagen se ve el ejemplo
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants