#7826 - Fix #1285

Merged
jsegarra merged 8 commits from 7826_fix into dev 2025-01-29 15:40:17 +00:00
4 changed files with 23 additions and 4 deletions

6
proxy-serve.js Normal file
View File

@ -0,0 +1,6 @@
export default [
{
path: '/api',
rule: { target: 'http://0.0.0.0:3000' },
},
];

View File

@ -179,7 +179,6 @@ export default configure(function (/* ctx */) {
'render', // keep this as last one
],
},
// https://v2.quasar.dev/quasar-cli/developing-pwa/configuring-pwa
pwa: {
workboxMode: 'generateSW', // or 'injectManifest'

View File

@ -20,6 +20,7 @@ const appName = 'Lilium';
const pinnedModulesRef = ref();
onMounted(() => stateStore.setMounted());
const refresh = () => window.location.reload();
</script>
<template>
<QHeader color="white" elevated>
@ -64,6 +65,13 @@ onMounted(() => stateStore.setMounted());
<QSpace />
<div class="q-pl-sm q-gutter-sm row items-center no-wrap">
<div id="actions-prepend"></div>
<QIcon
name="refresh"
size="md"
color="red"
v-if="state.get('error')"
@click="refresh"
/>
<QBtn
:class="{ 'q-pa-none': quasar.platform.is.mobile }"
id="pinnedModules"

View File

@ -21,8 +21,8 @@ const { t, te } = i18n.global;
const createHistory = process.env.SERVER
? createMemoryHistory
: process.env.VUE_ROUTER_MODE === 'history'
? createWebHistory
: createWebHashHistory;
? createWebHistory
: createWebHashHistory;
const Router = createRouter({
scrollBehavior: () => ({ left: 0, top: 0 }),
@ -99,9 +99,15 @@ export default defineRouter(function (/* { store, ssrContext } */) {
title += builtTitle;
}
document.title = title;
});
Router.onError(({ message }) => {
const errorMessages = [
'Failed to fetch dynamically imported module',
'Importing a module script failed',
];
state.set('error', errorMessages.some(message.includes));
});
Review

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
Review

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

Yo de momento pondria lo que habia en salix.

Yo de momento pondria lo que habia en salix.
return Router;
});