#7826 - Fix #1285

jsegarra · 2025-01-26T02:17:43Z

jsegarra commented

2025-01-26 02:17:43 +00:00

Para simular el icono sin necesidad de hacer el proceso desde terminal, basta con ir a este archivo src/router/index.js en la linea 102 y poner este codigo
setTimeout(() => {
state.set('error', true);
}, 5000);

Para probar el caso concreto que ocurre con los usuarios, seguir estos pasos

Para simular el icono sin necesidad de hacer el proceso desde terminal, basta con ir a este archivo src/router/index.js en la linea 102 y poner este codigo setTimeout(() => { state.set('error', true); }, 5000); Para probar el caso concreto que ocurre con los usuarios, seguir estos pasos

jsegarra added 4 commits 2025-01-26 02:17:43 +00:00

31a5dd653c fix: refs #7826 init

0c714d2c04 feat: refs #7826 add error handling and refresh icon to NavBar

94d471e20f Merge branch 'dev' into 7826_fix

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

24c62b4533 perf: refs #7826 improve condition

jsegarra added 1 commit 2025-01-26 22:49:45 +00:00

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

0aa9975535 perf: refs #7826 code onError

jsegarra requested review from jgallego 2025-01-26 22:50:05 +00:00

jsegarra requested review from alexm 2025-01-26 22:50:05 +00:00

jgallego approved these changes 2025-01-27 05:43:19 +00:00

Dismissed

alexm requested changes 2025-01-27 06:29:10 +00:00

Dismissed

src/router/index.js

						
				@ -106,0 +108,4 @@

				            'Importing a module script failed',

				        ];

				        state.set('error', errorMessages.some(message.includes));

				    });

alexm commented

2025-01-27 06:28:54 +00:00

Desde aquí no se podría lanzar un mensaje de error, y el mismo si aceptas te haga reload? @jgallego como lo ves?

quasar.dialog({
            component: VnConfirm,
            componentProps: {
                title: t('globals.-----'), // Nueva versión disponible
                message: t('globals.-----'), // Hay una nueva versión disponible. Si decides actualizar, perderás los datos no guardados. ¿Deseas continuar con la actualización?
                promise: () => window.location.reload
            },
        });
// En los botones que salga "cancelar" y "actualizar"
``
El mensaje no me convence

Desde aquí no se podría lanzar un mensaje de error, y el mismo si aceptas te haga reload? @jgallego como lo ves? ``` quasar.dialog({ component: VnConfirm, componentProps: { title: t('globals.-----'), // Nueva versión disponible message: t('globals.-----'), // Hay una nueva versión disponible. Si decides actualizar, perderás los datos no guardados. ¿Deseas continuar con la actualización? promise: () => window.location.reload }, }); // En los botones que salga "cancelar" y "actualizar" `` El mensaje no me convence

jsegarra commented

2025-01-27 08:50:23 +00:00

A falta de respuesta de javi, te cuento.

Yo tuve el mismo planteamiento al principio pero luego me di cuenta de estos 3 puntos.

Seria ponerle una lógica nueva al usuario,. En Salix estaba el icono
Al usar VnConfirm, no estoy 100% seguro de que funcione. Si modificas VnConfirm o por algún casual el hash de este archivo cambia, al montar el dialogo funcionará? Si funciona, funcionará el botón.?
Al mostrarle una notificación, aunque sea persistente, no se como les puede afectar ya que si pulsas en cancelar, mantendrías el error arrastrándolo a la siguiente ventana.

A falta de respuesta de javi, te cuento. Yo tuve el mismo planteamiento al principio pero luego me di cuenta de estos 3 puntos. 1. Seria ponerle una lógica nueva al usuario,. En Salix estaba el icono 2. Al usar VnConfirm, no estoy 100% seguro de que funcione. Si modificas VnConfirm o por algún casual el hash de este archivo cambia, al montar el dialogo funcionará? Si funciona, funcionará el botón.? 3. Al mostrarle una notificación, aunque sea persistente, no se como les puede afectar ya que si pulsas en cancelar, mantendrías el error arrastrándolo a la siguiente ventana.

jgallego commented

2025-01-27 11:33:03 +00:00

Yo de momento pondria lo que habia en salix.