Galeon2098 / CuidaME

Aplicación CuidaME para ISPP
2 stars 0 forks source link

Hotfix #260

Closed manueljgranados closed 3 months ago

manueljgranados commented 3 months ago

PR para despliegue de antes de Feria

codacy-production[bot] commented 3 months ago

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
:white_check_mark: +11.31% (target: -1.00%) :white_check_mark: 78.47%
Coverage variation details | | Coverable lines | Covered lines | Coverage | | ------------- | ------------- | ------------- | ------------- | | Common ancestor commit (86231779751d63d856f82bfce1902f16c81e7f52) | 1410 | 897 | 63.62% | | | Head commit (615bf239d313aff42af1b5f65a5caccaffe8506d) | 2074 (+664) | 1554 (+657) | 74.93% (**+11.31%**) | **Coverage variation** is the difference between the coverage for the head and common ancestor commits of the pull request branch: ` - `
Diff coverage details | | Coverable lines | Covered lines | Diff coverage | | ------------- | ------------- | ------------- | ------------- | | Pull request (#260) | 706 | 554 | **78.47%** | **Diff coverage** is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: `/ * 100%`

See your quality gate settings    Change summary preferences

You may notice some variations in coverage metrics with the latest Coverage engine update. For more details, visit the documentation

Lucperrom commented 3 months ago

Inicio: Imagen de inicio no responsive.

Detalles del perfil: -Cuando accedes a través de valoraciones al perfil de un cuidador no aparece la foto. -En la versión de móvil debería quitarse la imagen del anciano para que no ocupe tanto de la pantalla (opción móvil) -Navbar no responsive hay opciones que no se pueden pulsar

Solicitud de chat: -El cliente no tiene ninguna información acerca de su solicitud. Opción rápida: poner un mensaje en la parte superior de la pantalla que diga "Su solicitud se ha enviado correctamente". Opción lenta: en el listado de chats poner la solicitud como pendiente, que cuando se acepte la solicitud se cambie por el chat y cuando se deniegue aparezca un mensaje en su lugar diciendo "El cuidador ha rechazado su solicitud" con una "x" para cerrar el mensaje. Dado el momento en el que nos encontramos de la entrega recomiendo la primera opción

Detalles de oferta: -En la versión de móvil se ve muy fino

-Intereses: -Editar intereses no es responsive: al editar aparece una imagen que ocupa demasiada pantalla, y en el listado el recuadro no se ajusta al tamaño de la página (opción móvil)

-Editar perfil (tanto en cuidador como en cliente) en cliente se llama editar foto: -El navbar se mete detrás del cuadro de edición y no se pueden pulsar las opciones.

-Editar oferta: -Mismo problema que con intereses y detalles del perfil, aparece una imagen que ocupa la mayor parte de la pantalla en la opción móvil.

-Recordar en la notificación de oferta y en la verificación de correo electrónico cambiar la url por la del despliegue.

Respecto a la calidad: corregir el whitespace del test de selenium. Al respecto del sonarcloud los fallos de hotspot se deben todos a lo mismo usar la palabra "password" en los tests para loggearse, eso no se puede cambiar a menos que se hicies un cambio de nombre de ese atributo, cambio que no merece la pena dado que no es un fallo de seguridad.

sonarcloud[bot] commented 3 months ago

Quality Gate Failed Quality Gate failed

Failed conditions
50 Security Hotspots
17.7% Duplication on New Code (required ≤ 3%)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint