#8198 keep section filter #1210

Merged
jorgep merged 9 commits from 8198-keepSectionFilter into dev 2025-01-20 09:42:32 +00:00
Member
No description provided.
jorgep added 1 commit 2025-01-14 12:08:57 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
3cc5595e14
refactor: refs #8198 simplify data fetching and filtering logic
jorgep added 1 commit 2025-01-14 16:14:04 +00:00
jorgep added 2 commits 2025-01-16 09:40:54 +00:00
jorgep added 1 commit 2025-01-16 09:43:42 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
39adfb3693
fix: refs #8198 update query param
jorgep changed title from WIP: #8198 keep section filter to #8198 keep section filter 2025-01-16 09:51:27 +00:00
jorgep added 1 commit 2025-01-16 11:51:39 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
31795ae40b
test: refs #8198 skip tests
jorgep reviewed 2025-01-16 11:52:02 +00:00
@ -21,3 +21,3 @@
});
it('should refund the invoice ', () => {
it.skip('should refund the invoice ', () => {
Author
Member

Tarea para arreglar creada https://redmine.verdnatura.es/issues/8415

Tarea para arreglar creada https://redmine.verdnatura.es/issues/8415
jorgep added 1 commit 2025-01-16 15:27:22 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
b5ae3a1901
chore: refs #8198 rollback
jorgep reviewed 2025-01-16 15:27:39 +00:00
@ -107,3 +107,3 @@
name: 'WorkerList',
query: {
params: JSON.stringify({ departmentFk: entityId }),
table: JSON.stringify({ departmentFk: entityId }),
Author
Member

Lo arreglo de paso.

Lo arreglo de paso.
jorgep added 1 commit 2025-01-16 15:29:33 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
fb530d8408
chore: refs #8198 rollback
jorgep requested review from jsegarra 2025-01-16 15:30:11 +00:00
jorgep added the
CR / Tests passed
label 2025-01-16 15:30:17 +00:00
jsegarra requested changes 2025-01-16 16:01:20 +00:00
Dismissed
@ -48,3 +39,1 @@
where: where,
},
});
let arrayData = useArrayData('ItemLastEntries');
Member

porque let y no const?
Se modifica mas adelante y no lo veo?

porque let y no const? Se modifica mas adelante y no lo veo?
@ -181,2 +163,2 @@
_to.setDate(_to.getDate() + 10);
to.value = getDate(_to, 'to');
const landed = arrayData.store.filter.where?.landed;
arrayData = useArrayData('ItemLastEntries', {
Member

Se pone let porque aquí se machaca?
Y porque no lo hacemos fuera?
Quiero decir, de normal no hacemos esto de useArrayData dentro de onMounted

Se pone let porque aquí se machaca? Y porque no lo hacemos fuera? Quiero decir, de normal no hacemos esto de useArrayData dentro de onMounted
Author
Member

Es para que mantenga los filtros al cambiar de articulo desde la barra de búsqueda. Si lo hago fuera, antes de onMounted ya se habrá machacado el filter de la store. Lo que quiero es que al cambiar la url mantener el filtro que tenía.

Es para que mantenga los filtros al cambiar de articulo desde la barra de búsqueda. Si lo hago fuera, antes de onMounted ya se habrá machacado el filter de la store. Lo que quiero es que al cambiar la url mantener el filtro que tenía.
Member

Parece una chorrada, pero podemos hacer una constante que tenga este valor? porque ya hay que forzar la vista y hacer scroll arriba/abajo para ver si coinciden o no

Parece una chorrada, pero podemos hacer una constante que tenga este valor? porque ya hay que forzar la vista y hacer scroll arriba/abajo para ver si coinciden o no
jorgep requested review from jsegarra 2025-01-16 16:21:08 +00:00
Member

Te he respondido pero no sale como solicitar cambios

Te he respondido pero no sale como solicitar cambios
jsegarra approved these changes 2025-01-20 09:22:03 +00:00
jorgep added 1 commit 2025-01-20 09:30:39 +00:00
jorgep merged commit 2f40b77f0b into dev 2025-01-20 09:42:32 +00:00
jorgep deleted branch 8198-keepSectionFilter 2025-01-20 09:42:32 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: verdnatura/salix-front#1210
No description provided.