verdnatura/salix-front
verdnatura
/
salix-front
2
3
Fork 1
Code Issues Pull Requests 70 Releases Wiki Activity

#7826 - Fix #1285

Merged
jsegarra merged 8 commits from 7826_fix into dev 2025-01-29 15:40:17 +00:00
Conversation 0 Commits 8 Files Changed 4 +19 -2
2 changed files with 19 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 0c714d2c04 - Show all commits

8
src/components/NavBar.vue
View File

@ -63,7 +63,15 @@ onMounted(() => stateStore.setMounted());
<div id="searchbar" class="searchbar"></div> <div id="searchbar" class="searchbar"></div>
<QSpace /> <QSpace />
<div class="q-pl-sm q-gutter-sm row items-center no-wrap"> <div class="q-pl-sm q-gutter-sm row items-center no-wrap">
{{ state }}
<div id="actions-prepend"></div> <div id="actions-prepend"></div>
<QIcon
name="refresh"
size="md"
color="red"
v-if="state.get('error')"
@click="() => window.location.reload()"
/>
<QBtn <QBtn
:class="{ 'q-pa-none': quasar.platform.is.mobile }" :class="{ 'q-pa-none': quasar.platform.is.mobile }"
id="pinnedModules" id="pinnedModules"

13
src/router/index.js
View File

@ -15,6 +15,7 @@ import { useAcl } from 'src/composables/useAcl';
import { isLoggedIn } from 'src/utils/session'; import { isLoggedIn } from 'src/utils/session';
import { useSession } from 'src/composables/useSession'; import { useSession } from 'src/composables/useSession';
import { Notify } from 'quasar'; import { Notify } from 'quasar';
import { useStateStore } from 'stores/useStateStore';
let session = null; let session = null;
const { t, te } = i18n.global; const { t, te } = i18n.global;
@ -100,7 +101,9 @@ export default defineRouter(function (/* { store, ssrContext } */) {
title += builtTitle; title += builtTitle;
} }
setTimeout(() => {
state.set('error', true);
}, 1000);
document.title = title; document.title = title;
}); });
@ -108,13 +111,19 @@ export default defineRouter(function (/* { store, ssrContext } */) {
if ( if (
alexm commented 2025-01-27 06:28:54 +00:00
Review
Copy Link

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
Review
Copy Link

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.
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
Review
Copy Link

Yo de momento pondria lo que habia en salix.

Yo de momento pondria lo que habia en salix.
error.message.includes('Failed to fetch dynamically imported module') || error.message.includes('Failed to fetch dynamically imported module') ||
error.message.includes('Importing a module script failed') error.message.includes('Importing a module script failed')
) ) {
state.set('error', {
message: t('globals.errorLoadingPage'),
timeout: 5000,
type: 'negative',
});
Notify.create({ Notify.create({
message: t('globals.noSelectedRows'), message: t('globals.noSelectedRows'),
type: 'negative', type: 'negative',
timeout: 5000, timeout: 5000,
progress: true, progress: true,
}); });
}
}); });
return Router; return Router;
}); });