verdnatura/salix-front
verdnatura
/
salix-front
2
3
Fork 1
Code Issues Pull Requests 68 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 +12 -28
3 changed files with 12 additions and 28 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 24c62b4533 - Show all commits

6
src/boot/quasar.js
View File

@ -10,11 +10,7 @@ import { useCau } from 'src/composables/useCau';
export default boot(({ app }) => { export default boot(({ app }) => {
QForm.mixins = [qFormMixin]; QForm.mixins = [qFormMixin];
QLayout.mixins = [mainShortcutMixin]; QLayout.mixins = [mainShortcutMixin];
const metaVersion = process.env.BUILD_VERSION;
const storedVersion = localStorage.getItem('appVersion');
if (metaVersion !== storedVersion) {
localStorage.setItem('appVersion', metaVersion);
}
app.directive('shortcut', keyShortcut); app.directive('shortcut', keyShortcut);
app.config.errorHandler = async (error) => { app.config.errorHandler = async (error) => {
let message; let message;

3
src/components/NavBar.vue
View File

@ -20,6 +20,7 @@ const appName = 'Lilium';
const pinnedModulesRef = ref(); const pinnedModulesRef = ref();
onMounted(() => stateStore.setMounted()); onMounted(() => stateStore.setMounted());
const refresh = () => window.location.reload();
</script> </script>
<template> <template>
<QHeader color="white" elevated> <QHeader color="white" elevated>
@ -71,7 +72,7 @@ onMounted(() => stateStore.setMounted());
size="md" size="md"
color="red" color="red"
v-if="state.get('error')" v-if="state.get('error')"
@click="() => window.location.reload()" @click="refresh"
/> />
<QBtn <QBtn
:class="{ 'q-pa-none': quasar.platform.is.mobile }" :class="{ 'q-pa-none': quasar.platform.is.mobile }"

31
src/router/index.js
View File

@ -14,8 +14,6 @@ import { useTokenConfig } from 'src/composables/useTokenConfig';
import { useAcl } from 'src/composables/useAcl'; 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 { useStateStore } from 'stores/useStateStore';
let session = null; let session = null;
const { t, te } = i18n.global; const { t, te } = i18n.global;
@ -101,29 +99,18 @@ export default defineRouter(function (/* { store, ssrContext } */) {
title += builtTitle; title += builtTitle;
} }
setTimeout(() => {
state.set('error', true);
}, 1000);
document.title = title; document.title = title;
}); });
Router.onError((error, to) => { Router.onError((error) => {
if ( const errorMessages = [
error.message.includes('Failed to fetch dynamically imported module') || 'Failed to fetch dynamically imported module',
error.message.includes('Importing a module script failed') 'Importing a module script failed',
) { ];
state.set('error', { state.set(
message: t('globals.errorLoadingPage'), 'error',
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.
timeout: 5000, errorMessages.some((msg) => error.message.includes(msg)),
type: 'negative', );
});
Notify.create({
message: t('globals.noSelectedRows'),
type: 'negative',
timeout: 5000,
progress: true,
});
}
}); });
return Router; return Router;
}); });