hotfix: fix parsePhone fn when not phone available #1028

jsegarra · 2024-12-01T18:58:31Z

jsegarra commented

2024-12-01 18:58:31 +00:00

Se ha detectado que con ciertos usuarios de producción aparece un error al cargar el summary
Por ejemplo, para el usuario 34655, cuando carga el summary da error, y no es correcto.
Aparece la notificación de error porque no tiene phone.

Ya que corregimos esto, aplico otro cambio que nos pidieron de abrirlo en una nueva ventana. He comprobado que las URLs coincidan.

Si detectas que no es correcto, quitamos el commit o lo hablamos.

Al menos Paco Rico nos lo agradecerá

Había test unitario que he modificado para incluir el caso null pero no veo ningún e2e

Se ha detectado que con ciertos usuarios de producción aparece un error al cargar el summary Por ejemplo, para el usuario 34655, cuando carga el summary da error, y no es correcto. Aparece la notificación de error porque no tiene phone. Ya que corregimos esto, aplico otro cambio que nos pidieron de abrirlo en una nueva ventana. He comprobado que las URLs coincidan. Si detectas que no es correcto, quitamos el commit o lo hablamos. Al menos Paco Rico nos lo agradecerá Había test unitario que he modificado para incluir el caso null pero no veo ningún e2e

jsegarra added 2 commits 2024-12-01 18:58:32 +00:00

e2304396d8 fix: parsePhone fn when no phone

gitea/salix-front/pipeline/pr-master This commit looks good Details

137798c959 perf: useOpenURL SaySimple

jsegarra requested review from jorgep 2024-12-01 19:00:40 +00:00

jorgep closed this pull request

2024-12-02 09:37:44 +00:00

jorgep commented

2024-12-02 09:38:05 +00:00

Añadido en esta rama: #1013

Añadido en esta rama: https://gitea.verdnatura.es/verdnatura/salix-front/pulls/1013

jorgep deleted branch hotfix_parsePhone