#7308 - Warning inject #1065

Merged
jsegarra merged 10 commits from 7308_warning into dev 2025-01-22 10:41:33 +00:00
1 changed files with 3 additions and 1 deletions
Showing only changes of commit 7f281b8630 - Show all commits

View File

@ -15,6 +15,7 @@ import { useAcl } from 'src/composables/useAcl';
import { isLoggedIn } from 'src/utils/session';
import { useSession } from 'src/composables/useSession';
let session = null;
const { t, te } = i18n.global;
const createHistory = process.env.SERVER
@ -45,7 +46,8 @@ export { Router };
export default defineRouter(function (/* { store, ssrContext } */) {
Review

El dejarlo fuera del Router.beforeEach, solo lo llamarà una vez entiendo, queremos ese enfoque?

El dejarlo fuera del Router.beforeEach, solo lo llamarà una vez entiendo, queremos ese enfoque?
Review

El comentario es sobre el useState?
Lo pregunto porque este, al que me refiero, ya estaba fuera del beforeEach, en la linea 17

El comentario es sobre el useState? Lo pregunto porque este, al que me refiero, ya estaba fuera del beforeEach, en la linea 17
Review

Sisi por eso, ahora esta dentro, por lo que en cada iteración del router se llamará. Como estaba antes entiendo q solo se llamará 1 vez

Sisi por eso, ahora esta dentro, por lo que en cada iteración del router se llamará. Como estaba antes entiendo q solo se llamará 1 vez
Review

solved

solved
const state = useState();
Router.beforeEach(async (to, from, next) => {
const session = useSession();
console.error('beforeEach', session);
Review

Quitar entiendo

Quitar entiendo
if (!session) session = useSession();
const outLayout = Router.options.routes[0].children.map((r) => r.name);
if (!session.isLoggedIn() && !outLayout.includes(to.name)) {
return next({ name: 'Login', query: { redirect: to.fullPath } });