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

#7308 - Warning inject #1065

Merged
jsegarra merged 10 commits from 7308_warning into dev 2025-01-22 10:41:33 +00:00
Conversation 3 Commits 10 Files Changed 5 -1
1 changed files with 0 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 2d8dd46d15 - Show all commits

1
src/router/index.js
View File

@ -46,7 +46,6 @@ export { Router };
export default defineRouter(function (/* { store, ssrContext } */) {
alexm commented 2025-01-03 06:48:59 +00:00
Review
Copy Link

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?
jsegarra commented 2025-01-03 16:09:31 +00:00
Review
Copy Link

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
alexm commented 2025-01-08 06:28:28 +00:00
Review
Copy Link

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
jsegarra commented 2025-01-13 11:30:16 +00:00
Review
Copy Link

solved

solved
const state = useState();
Router.beforeEach(async (to, from, next) => {
console.error('beforeEach', session);
if (!session) session = useSession();
alexm commented 2025-01-13 12:08:04 +00:00
Review
Copy Link

Quitar entiendo

Quitar entiendo
const outLayout = Router.options.routes[0].children.map((r) => r.name);
if (!session.isLoggedIn() && !outLayout.includes(to.name)) {