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

MONIT - Actualizacion de ABM en novedades #62

Merged
merged 1 commit into from
Feb 29, 2024
Merged

MONIT - Actualizacion de ABM en novedades #62

merged 1 commit into from
Feb 29, 2024

Conversation

ma7payne
Copy link
Contributor

@ma7payne ma7payne commented Feb 8, 2023

Requerimiento

https://proyectos.andes.gob.ar/browse/MONIT-52

Funcionalidad desarrollada

  1. Incorpora campo para agregar una palabra significativa para la novedad
  2. Limita tamaño de imágenes a 5MB
  3. Muestra mensaje de error al superar limite de 5MB

UserStory llegó a completarse

  • Si
  • No
  • No corresponde

Requiere actualizaciones en la base de datos

  • Si
  • No

Requiere actualizaciones en la API

Link PR: andes/api#1784

  • Si
  • No

Requiere actualizaciones en andes-test-integracion

  • Si
  • No

@MarianoCampetella
Copy link
Contributor

Maty encontre un comportamiento raro que en demo no me sucede! Cuando abro el sidebar y hago un scroll del listado, no solamente me muestra mas datos en dicho listado (revisar si este comportamiento esta bien) pero lo que si no es correcto es que me cierre el sidebar. Ya que estamos aprovecho a decirte que tenes algunos errores de linteo para solucionar.

Andes._.Monitoreo.-.14.February.2023.mp4

@ma7payne
Copy link
Contributor Author

ma7payne commented Sep 5, 2023

Maty encontre un comportamiento raro que en demo no me sucede! Cuando abro el sidebar y hago un scroll del listado, no solamente me muestra mas datos en dicho listado (revisar si este comportamiento esta bien) pero lo que si no es correcto es que me cierre el sidebar. Ya que estamos aprovecho a decirte que tenes algunos errores de linteo para solucionar.

Andes._.Monitoreo.-.14.February.2023.mp4

Hola, respecto a ese comportamiento parece que lo heredó en el momento en que hice el branch, nada en los cambios actuales afecta el funcionamiento del sidebar. Deberiamos dejarlo como nueva tarea.

@silviroa
Copy link
Contributor

silviroa commented Sep 5, 2023

USUARIO: mchiappe
BUILD NUMBER: 9556
CYPRESS RUN: 8679
TEST START: 2023-09-05T14:19:11.007Z
TOTAL: 403
SUCCESS: 401
FAIL: 0
SKIPPED: 2

@MarianoCampetella
Copy link
Contributor

Se debe crear una nueva tarea para fixear el problema mencionado anteriormente!

@ma7payne ma7payne added changes done Se realizaron los cambios pedidos. and removed changes requested Se solicitaron cambios labels Oct 2, 2023
@maring0019 maring0019 self-requested a review October 3, 2023 16:41
Copy link
Contributor

@maring0019 maring0019 left a comment

Choose a reason for hiding this comment

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

Sugiero cambiar el valor de comparación de los 5MB. Está escrito como 5000000 pero el valor exacto es 5242880. Ya que sino no aceptaría imágenes que si son menores a 5MB.

Me surgió una consulta. ¿Y lo del límite de 5MB por qué? ¿Por qué no otro valor?

@maring0019 maring0019 added changes requested Se solicitaron cambios and removed changes done Se realizaron los cambios pedidos. labels Oct 3, 2023
@ma7payne
Copy link
Contributor Author

ma7payne commented Oct 3, 2023

Sugiero cambiar el valor de comparación de los 5MB. Está escrito como 5000000 pero el valor exacto es 5242880. Ya que sino no aceptaría imágenes que si son menores a 5MB.

Me surgió una consulta. ¿Y lo del límite de 5MB por qué? ¿Por qué no otro valor?

Listo el cambio!

@maring0019 maring0019 self-requested a review October 4, 2023 17:36
@maring0019 maring0019 added changes done Se realizaron los cambios pedidos. and removed changes requested Se solicitaron cambios labels Oct 4, 2023
@silviroa
Copy link
Contributor

silviroa commented Oct 4, 2023

USUARIO: mchiappe
BUILD NUMBER: 9737
CYPRESS RUN: 8848
TEST START: 2023-10-04T19:11:29.547Z
TOTAL: 403
SUCCESS: 401
FAIL: 0
SKIPPED: 2

@silviroa silviroa removed the test ok label Oct 4, 2023
@negro89 negro89 merged commit 361ca2a into master Feb 29, 2024
1 check passed
@negro89 negro89 deleted the MONIT-52 branch February 29, 2024 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes done Se realizaron los cambios pedidos. test ok
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants