Galeon2098 / CuidaME

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

Hotfix #303

Closed manueljgranados closed 5 months ago

manueljgranados commented 5 months ago

Tunning S3

codacy-production[bot] commented 5 months ago

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
:white_check_mark: +5.20% (target: -1.00%) :white_check_mark: 69.03%
Coverage variation details | | Coverable lines | Covered lines | Coverage | | ------------- | ------------- | ------------- | ------------- | | Common ancestor commit (1d922a1d4539453a48c678c5b9f795cdb68244af) | 2346 | 1665 | 70.97% | | | Head commit (c8449b6b1c39b360e87305e3377d728402b23f4b) | 3076 (+730) | 2343 (+678) | 76.17% (**+5.20%**) | **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 (#303) | 749 | 517 | **69.03%** | **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

Codacy will stop sending the deprecated coverage status from June 5th, 2024. Learn more

Lucperrom commented 5 months ago

Comentarios pre-revisión:

Lucperrom commented 5 months ago

Pantallazo de error si entras en listado de ofertas sin estar loggeado. Por lo que parece no hay ofertas creadas y peta porque intenta recorrer un null.

image
Lucperrom commented 5 months ago
image

Este botón para loggearse no es intuitivo. Se suele utilizar para desloggearse, e igualmente dudo que sea intuitivo para una persona mayor ese botón para cerrar sesión. Contrastar con algún usuario piloto antes de hacer cambios de interfaz. Volver a cómo se veía anteriormente.

Lucperrom commented 5 months ago

Dónde está el loggin con google?

image
Lucperrom commented 5 months ago

La portada no es responsive:

image
codacy-production[bot] commented 5 months ago

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
:white_check_mark: +5.20% (target: -1.00%) :white_check_mark: 69.03%
Coverage variation details | | Coverable lines | Covered lines | Coverage | | ------------- | ------------- | ------------- | ------------- | | Common ancestor commit (1d922a1d4539453a48c678c5b9f795cdb68244af) | 2346 | 1665 | 70.97% | | | Head commit (30c4a3a05885ba55c085a707cdd06ef73deb5686) | 3076 (+730) | 2343 (+678) | 76.17% (**+5.20%**) | **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 (#303) | 749 | 517 | **69.03%** | **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

Codacy will stop sending the deprecated coverage status from June 5th, 2024. Learn more

fausto4VR commented 5 months ago

Portada ha sido rehecha de forma más responsive: Captura4

Listado de ofertas sin estar logueado es accesible hasta cuando no existe ninguna. A lo mejor ha sido un error al hacer las migraciones, no se. Simplemente le he dado al botón de listado de ofertas y ha funcionado: Captura3

Iniciar sesión con Google ha sido vuelto a poner: Captura2

El botón de iniciar sesión ha sido cambiado a uno más intuitivo: Captura

joaquin-restoy commented 5 months ago

Falta navbar en calendario: image

fausto4VR commented 5 months ago

La barra de navegación en el calendario creo que me dijo el chaval que lo hizo que le fue imposible de poner. Es de las pocas cosas que el fix de frontend no ha podido revisar porque se ha incorporado hace nada

joaquin-restoy commented 5 months ago

Al registrarse con el formulario incluyendo el email y después hacer login con cuenta de google de ese mismo correo sale lo siguiente: image

Si recargas aparece lo siguiente: image

Lucperrom commented 5 months ago

Es muy difícil encontrar una dirección con la que te deje publicar una oferta. Mejor no poner tantas restricciones. Esto solo ocurre al crearla, al editarla sin embargo sí me deja algunas direcciones que antes no. Es preferible que el mapa la "ubique mal" en un principio que el cuidador lo vea y lo edite a que no le deje crearla directamente.

image
Lucperrom commented 5 months ago

Sé que la intención es que los botones ocupen menos del navbar, pero esto ya se llegó a cambiar una vez porque precios no resulta intuitivo, si lo quieren acortar mejor "planes" o "suscripciones".

image
Lucperrom commented 5 months ago

Cuando se pone en formato móvil o tablet quitar la imagen. Ocupa la mayoría de la pantalla. Esto ocurre tanto en cuidador como en cliente

image
fausto4VR commented 5 months ago

Se ha cambiado el botón de precios a suscripciones: Captura

Se ha hecho que las pantallas menores a 992px (moviles y algunas tablets) no se muestre la imagen de perfil: Captura2

codacy-production[bot] commented 5 months ago

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
:white_check_mark: +5.18% (target: -1.00%) :white_check_mark: 68.71%
Coverage variation details | | Coverable lines | Covered lines | Coverage | | ------------- | ------------- | ------------- | ------------- | | Common ancestor commit (1d922a1d4539453a48c678c5b9f795cdb68244af) | 2346 | 1665 | 70.97% | | | Head commit (1c63652387e3fea4c868e6413a7204cd8f5cb5a2) | 3074 (+728) | 2341 (+676) | 76.15% (**+5.18%**) | **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 (#303) | 751 | 516 | **68.71%** | **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

Codacy will stop sending the deprecated coverage status from June 5th, 2024. Learn more

fausto4VR commented 5 months ago

También se ha probado a crear una oferta y el campo de dirección no ha dado errores si se ponía una que existiera dentro de la población. Es posible que haya demasiadas restricciones en ese sentido, pero el error de ayer (que me saltó también) fue un 403 y creo que era por algún tema de que Google Maps (o lo que se use) no estaba procesando las solicitudes. No se si porque estaba caído, porque habíamos hecho demasiadas solicitudes o si estábamos haciendo mal la solicitud por alguna razón. Hoy ya si me ha dejado crear la oferta sin problemas.