#7308 - Warning inject #1065

Al separarlo, se han duplicado las variables de tokenMultimedia. Me suena q hicimos un composable o store para las variables que siempre eran iguales

Al separarlo, se han duplicado las variables de `tokenMultimedia`. Me suena q hicimos un composable o store para las variables que siempre eran iguales

alexm reviewed 2025-01-03 06:46:50 +00:00

src/router/index.js Outdated

						
				@ -44,3 +43,3 @@

				 */

				export { Router };

				export default route(function (/* { store, ssrContext } */) {

				export default defineRouter(function (/* { store, ssrContext } */) {

alexm commented

2025-01-03 06:46:50 +00:00

Que cambia del export default route al export default defineRouter ¿?

Que cambia del `export default route` al `export default defineRouter` ¿?

jsegarra commented

2025-01-03 16:08:13 +00:00

Esto lo vi en la documentación oficial, pero ahora no recuerdo el enlace exacto. Lo buscaré

jsegarra commented

2025-01-13 11:31:14 +00:00

#1065 (comment)

https://gitea.verdnatura.es/verdnatura/salix-front/pulls/1065#issuecomment-74021

alexm reviewed 2025-01-03 06:48:59 +00:00

src/router/index.js

						
				@ -45,2 +44,3 @@

				export { Router };

				export default route(function (/* { store, ssrContext } */) {

				export default defineRouter(function (/* { store, ssrContext } */) {

				    const state = useState();

alexm commented

2025-01-03 06:48:59 +00:00

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

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

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

solved

jsegarra commented

2025-01-03 16:10:21 +00:00

Al separarlo, se han duplicado las variables de tokenMultimedia. Me suena q hicimos un composable o store para las variables que siempre eran iguales

Lo reviso, porque es cierto que en la *Storage no he mirado, entiendo que va por clave/key

> Al separarlo, se han duplicado las variables de `tokenMultimedia`. Me suena q hicimos un composable o store para las variables que siempre eran iguales Lo reviso, porque es cierto que en la *Storage no he mirado, entiendo que va por clave/key

jsegarra requested review from alexm 2025-01-07 14:56:21 +00:00

alexm requested changes 2025-01-08 06:29:31 +00:00

Dismissed

alexm left a comment

Revisar lo del defineRouter

jsegarra commented

2025-01-08 17:06:43 +00:00

Revisar lo del defineRouter

Al fin lo encontré, https://quasar.dev/quasar-cli-vite/routing#introduction

> Revisar lo del defineRouter Al fin lo encontré, https://quasar.dev/quasar-cli-vite/routing#introduction ![image](/attachments/72b34876-721f-4ce9-80b1-36fd9ceb728a)

image.png

85 KiB

jsegarra added 2 commits 2025-01-13 11:30:14 +00:00

70000e3eca Merge branch 'dev' into 7308_warning

gitea/salix-front/pipeline/pr-dev This commit looks good Details

7f281b8630 perf: refs #7308 call 1 time useSession

jsegarra requested review from alexm 2025-01-13 11:30:23 +00:00

alexm requested changes 2025-01-13 12:08:06 +00:00

Dismissed

src/router/index.js

						
				@ -47,2 +47,3 @@

				    const state = useState();

				    Router.beforeEach(async (to, from, next) => {

				        const { isLoggedIn } = session;

				        console.error('beforeEach', session);