verdnatura/salix
verdnatura
/
salix
3
4
Fork 0
Code Issues Pull Requests 57 Releases Wiki Activity

#7272 - BUG Redirect To Login when ACL not exists #2371

Merged
jsegarra merged 19 commits from 7272_bug_redirectToLogin into dev 2024-05-27 06:56:20 +00:00
Conversation 3 Commits 19 Files Changed 7 +78 -26
jsegarra commented 2024-04-25 11:07:17 +00:00
Member
Copy Link
No description provided.
jsegarra added 1 commit 2024-04-25 11:17:03 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
5bbf5887b9 fix(salix): refs #7272 #7272 Bug when acl not loaded
jsegarra requested review from juan 2024-04-25 11:19:44 +00:00
jsegarra requested review from jgallego 2024-04-25 11:19:44 +00:00
jsegarra requested review from alexm 2024-04-25 11:19:44 +00:00
jgallego approved these changes 2024-04-25 12:35:21 +00:00
alexm reviewed 2024-04-25 13:02:18 +00:00
front/core/services/auth.js Outdated
@ -44,1 +44,3 @@
.catch(redirectToLogin);
const maxRetries = 5;
let retryCount = 0;
const retryDelay = 2000; // Milisegundos (1 segundo)
alexm commented 2024-04-25 13:02:18 +00:00
Member
Copy Link

2 no? jajaj

2 no? jajaj
jsegarra commented 2024-04-25 13:03:51 +00:00
Author
Member
Copy Link

5 intentos máximos cada 2 segundos

He respondido a tu pregunta? @alexm

5 intentos máximos cada 2 segundos He respondido a tu pregunta? @alexm
alexm requested changes 2024-04-26 06:08:39 +00:00
Dismissed
front/core/services/auth.js Outdated
@ -126,0 +151,4 @@
} else {
// Retry after delay
return new Promise((resolve, reject) => {
reject(new Error('Error al cargar los ACLs'));
alexm commented 2024-04-26 06:08:06 +00:00
Member
Copy Link

Traducir y tampoco le diria al usuario un mensaje tan especifico

Traducir y tampoco le diria al usuario un mensaje tan especifico
jsegarra commented 2024-04-26 06:16:26 +00:00
Author
Member
Copy Link

Este mensaje es transparente para el usuario. No se muestra por pantalla
El unico toast que aparece es el de "Cargando..."

Este mensaje es transparente para el usuario. No se muestra por pantalla El unico toast que aparece es el de "Cargando..."
alexm commented 2024-04-29 05:40:43 +00:00
Member
Copy Link

Traducir igualmente

Traducir igualmente
jsegarra requested review from alexm 2024-04-26 17:33:12 +00:00
jsegarra changed title from BUG Redirect To Login when ACL not exists to #7272 - BUG Redirect To Login when ACL not exists 2024-04-26 19:08:27 +00:00
jsegarra added 2 commits 2024-04-29 10:51:28 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
cf800ae6b4 Merge branch 'dev' into 7272_bug_redirectToLogin
jsegarra changed title from #7272 - BUG Redirect To Login when ACL not exists to WIP: #7272 - BUG Redirect To Login when ACL not exists 2024-04-30 05:57:29 +00:00
jsegarra commented 2024-04-30 05:57:58 +00:00
Author
Member
Copy Link

Se cambia a WIP, ver https://redmine.verdnatura.es/issues/7272#note-6

Se cambia a WIP, ver https://redmine.verdnatura.es/issues/7272#note-6
jsegarra added 1 commit 2024-04-30 05:58:06 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
60634fc308 Merge branch 'dev' into 7272_bug_redirectToLogin
jsegarra added 7 commits 2024-04-30 09:46:27 +00:00
jsegarra added 1 commit 2024-05-02 07:41:29 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
bc4c671587 Merge branch 'dev' into 7272_bug_redirectToLogin
jsegarra added 3 commits 2024-05-02 08:37:20 +00:00
jsegarra changed title from WIP: #7272 - BUG Redirect To Login when ACL not exists to #7272 - BUG Redirect To Login when ACL not exists 2024-05-02 10:44:09 +00:00
alexm requested changes 2024-05-03 05:17:55 +00:00
Dismissed
alexm left a comment
Member
Copy Link

No funciona, al hacer gulp muestra errores:
image

Me he ido a herramientas de desarrollo y al borrarme el token ya me ha dejado

No funciona, al hacer gulp muestra errores: ![image](/attachments/cf1b3cbe-74a9-43c5-bbf9-f911258529f6) Me he ido a herramientas de desarrollo y al borrarme el token ya me ha dejado
image.png
77 KiB
jsegarra added 1 commit 2024-05-03 06:25:05 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
da3c4cb8fc fix(salix): refs #7272 #7272 Errors when Token not exists
jsegarra requested review from alexm 2024-05-03 06:26:30 +00:00
jsegarra commented 2024-05-03 06:37:06 +00:00
Author
Member
Copy Link

No funciona, al hacer gulp muestra errores:
image

Me he ido a herramientas de desarrollo y al borrarme el token ya me ha dejado

Esta prueba no la habia reproducido.

Tienes un token en base de datos
Ejecuta myt run
El token se borra
El token portanto, desde el front no es valido.
ERROR No redirige

Ya está corregido y los e2e OK

> No funciona, al hacer gulp muestra errores: > ![image](/attachments/cf1b3cbe-74a9-43c5-bbf9-f911258529f6) > > Me he ido a herramientas de desarrollo y al borrarme el token ya me ha dejado Esta prueba no la habia reproducido. Tienes un token en base de datos Ejecuta myt run El token se borra El token portanto, desde el front no es valido. ERROR No redirige Ya está corregido y los e2e OK
juan requested changes 2024-05-03 07:35:26 +00:00
Dismissed
juan left a comment
Owner
Copy Link
  • Mover la descarga de ACLs i carga de modulos al resolve del estado layout en el fichero front/salix/routes.js.
  • Añadir tambien al resolve una petición para validar el token.
  • No escribir código para reintentar peticiones fallidas.
- Mover la descarga de ACLs i carga de modulos al `resolve` del estado `layout` en el fichero _front/salix/routes.js_. - Añadir tambien al resolve una petición para validar el token. - No escribir código para reintentar peticiones fallidas.
jsegarra changed title from #7272 - BUG Redirect To Login when ACL not exists to WIP: #7272 - BUG Redirect To Login when ACL not exists 2024-05-03 08:34:45 +00:00
jsegarra added 1 commit 2024-05-03 10:27:10 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
6e00b790d2 Merge branch 'dev' into 7272_bug_redirectToLogin
jsegarra commented 2024-05-03 10:30:12 +00:00
Author
Member
Copy Link
  • Mover la descarga de ACLs i carga de modulos al resolve del estado layout en el fichero front/salix/routes.js.
  • Añadir tambien al resolve una petición para validar el token.
  • No escribir código para reintentar peticiones fallidas.

Hola @juan

  1. El primer punto ya estaba hecho un funcionando. De echo, como está en la PR se están cargando los 16 modulos y las acls correctamente.
  2. El segundo punto, tal y como hemos comentado, ya estaba en la PR
  3. He quitado el reintento de petición fallida. Ahora, si te devuelve al login, al menos que te diga porque ha sido
> - Mover la descarga de ACLs i carga de modulos al `resolve` del estado `layout` en el fichero _front/salix/routes.js_. > - Añadir tambien al resolve una petición para validar el token. > - No escribir código para reintentar peticiones fallidas. Hola @juan 1. El primer punto ya estaba hecho un funcionando. De echo, como está en la PR se están cargando los 16 modulos y las acls correctamente. 2. El segundo punto, tal y como hemos comentado, ya estaba en la PR 3. He quitado el reintento de petición fallida. Ahora, si te devuelve al login, al menos que te diga porque ha sido
jsegarra changed title from WIP: #7272 - BUG Redirect To Login when ACL not exists to #7272 - BUG Redirect To Login when ACL not exists 2024-05-03 10:30:13 +00:00
jsegarra requested review from juan 2024-05-03 10:30:17 +00:00
alexm approved these changes 2024-05-06 06:06:19 +00:00
jgallego removed review request for juan 2024-05-24 11:35:57 +00:00
jgallego dismissed juan’s review 2024-05-24 11:36:02 +00:00
Reason:

expired

jsegarra added 1 commit 2024-05-24 11:59:33 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
ab1797ebec Merge branch 'dev' into 7272_bug_redirectToLogin
jsegarra added 1 commit 2024-05-27 06:54:58 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
693356f4cb Merge branch 'dev' into 7272_bug_redirectToLogin
jsegarra merged commit 50ef375c6b into dev 2024-05-27 06:56:20 +00:00
jsegarra deleted branch 7272_bug_redirectToLogin 2024-05-27 06:56:20 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
jgallego
alexm
Labels
Clear labels   
CR / Tests passed

Unit tests and e2e passed

  
e2e

Implement e2e

  
Missing tests

Missing tests

  
Needs refactor

   
Simple

Simple check / No CR needed

No Label
CR / Tests passed
e2e
Missing tests
Needs refactor
Simple
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
4 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: verdnatura/salix#2371
No description provided.
Delete Branch "7272_bug_redirectToLogin"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?