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

perf_redirectTransition #1178

Merged
alexm merged 7 commits from perf_redirectTransition into test 2025-01-09 08:16:54 +00:00
Conversation 0 Commits 7 Files Changed 1 +1 -2
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 2505061b8c - Show all commits

3
src/composables/useArrayData.js
View File

@ -125,8 +125,7 @@ export function useArrayData(key = useRoute().meta.moduleName, userOptions) {
store.hasMoreData = limit && response.data.length >= limit;
if (!append && !isDialogOpened() && updateRouter) {
alexm commented 2025-01-07 07:25:26 +00:00
Review
Copy Link

Basicamente es hacer primero la parte del router, antes de procesar los datos. Y si se va a redirigir evitamos la parte de setear datos

Basicamente es hacer primero la parte del router, antes de procesar los datos. Y si se va a redirigir evitamos la parte de setear datos
jsegarra commented 2025-01-07 13:25:17 +00:00
Review
Copy Link

Estamos en el limite de condiciones de un if

Estamos en el limite de condiciones de un if
👀 1
const res = updateStateParams(response.data);
if (res?.redirect) return;
if (updateStateParams()?.redirect) return;
jsegarra commented 2025-01-07 13:44:49 +00:00
Outdated
Review
Copy Link

Ya tenemos un response por ahi, asi que la variable res, se puede renombrar?
Y si hacemos const redirect = updateStateParams()?.redirect?
if(redirect) return?

Es una idea, por eso lo pongo como comentario

Ya tenemos un **res**ponse por ahi, asi que la variable res, se puede renombrar? Y si hacemos const redirect = updateStateParams()?.redirect? if(redirect) return? Es una idea, por eso lo pongo como comentario
alexm commented 2025-01-09 06:57:26 +00:00
Outdated
Review
Copy Link

Me mola, si. Lo subo asi

Me mola, si. Lo subo asi
❤️ 1
}
store.isLoading = false;
canceller = null;