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

warmifx: Customer_filters #1131

Merged
alexm merged 10 commits from warmifx_customer_filters into test 2025-01-28 06:17:31 +00:00
Conversation 4 Commits 10 Files Changed 2 +54 -56
jsegarra commented 2024-12-24 11:17:05 +00:00
Member
Copy Link

@alexm Recuerdas porque dejamos las columnas de CustomerExtendedList pero la url no la cambiamos?
Creo que es por eso por lo que muchas columnas aparecen con - y los filtros no van

https://cau.verdnatura.es/scp/tickets.php?id=255062#ticket_thread

verdnatura/salix#3384

@alexm Recuerdas porque dejamos las columnas de CustomerExtendedList pero la url no la cambiamos? Creo que es por eso por lo que muchas columnas aparecen con - y los filtros no van https://cau.verdnatura.es/scp/tickets.php?id=255062#ticket_thread https://gitea.verdnatura.es/verdnatura/salix/pulls/3384
jsegarra added 2 commits 2024-12-24 11:17:05 +00:00
gitea/salix-front/pipeline/pr-test There was a failure building this commit Details
2c86fb17f1 fix: fix some filters
jsegarra requested review from alexm 2024-12-24 11:17:09 +00:00
alexm requested changes 2025-01-03 06:40:34 +00:00
alexm left a comment
Member
Copy Link

Si se puede subir esto despues de #

Si se puede subir esto despues de #
src/pages/Customer/CustomerList.vue Outdated
@ -434,0 +442,4 @@
:title="title"
/>
</template>
<template #option="scope">
alexm commented 2025-01-03 06:39:32 +00:00
Member
Copy Link

Esta parte ya la tiene o la tendra por defecto, VnSelect mirar de quitar

Esta parte ya la tiene o la tendra por defecto, VnSelect mirar de quitar
👍 1
jsegarra commented 2025-01-03 16:05:30 +00:00
Author
Member
Copy Link

Cierto, hice, literal, copy paste

Cierto, hice, literal, copy paste
jsegarra commented 2025-01-04 09:09:01 +00:00
Author
Member
Copy Link

Estoy revisando y copie justo la unica instancia de VnSelectWorker con las options definidas

Estoy revisando y copie justo la unica instancia de VnSelectWorker con las options definidas
jsegarra commented 2025-01-04 09:11:47 +00:00
Author
Member
Copy Link

@jon Me puedes confirmar que en CustomerList que es de donde se copio este código es necesario mantener las options? O se coló?
Gracias de antemano.

Si se coló, yo creo que podemos hacer el "fix" en esta misma rama

@jon Me puedes confirmar que en CustomerList que es de donde se copio este código es necesario mantener las options? O se coló? Gracias de antemano. Si se coló, yo creo que podemos hacer el "fix" en esta misma rama
jon commented 2025-01-04 14:29:44 +00:00
Member
Copy Link

Si no me equivoco el componente VnSelectWorker ya tiene las options para el avatar, solo hay que pasarle la prop a true para que te lo muestre, si no me equivoco ya hay un sitio donde se hizo así

Si no me equivoco el componente VnSelectWorker ya tiene las options para el avatar, solo hay que pasarle la prop a true para que te lo muestre, si no me equivoco ya hay un sitio donde se hizo así
jsegarra commented 2025-01-07 13:21:41 +00:00
Author
Member
Copy Link

por tanto, habría que eliminarlo de CustomerList.
Okey, gracias

por tanto, habría que eliminarlo de CustomerList. Okey, gracias
jsegarra commented 2025-01-27 13:03:53 +00:00
Author
Member
Copy Link

resuelto

resuelto
jsegarra commented 2025-01-25 13:50:58 +00:00
Author
Member
Copy Link

Si se puede subir esto después de #

Se ha perdido la referencia de la PR

Ahora la url está cambiada, pero los filtros hay que revisarlo

> Si se puede subir esto después de # Se ha perdido la referencia de la PR Ahora la url está cambiada, pero los filtros hay que revisarlo
jsegarra added 2 commits 2025-01-27 12:52:48 +00:00
gitea/salix-front/pipeline/pr-test There was a failure building this commit Details
d1ba6e580f fix: params
jsegarra added 1 commit 2025-01-27 13:03:44 +00:00
gitea/salix-front/pipeline/pr-test There was a failure building this commit Details
9dd216269a fix: vnSelectWorker fix template
jsegarra added 2 commits 2025-01-27 13:09:03 +00:00
gitea/salix-front/pipeline/pr-test This commit looks good Details
8cbb97af29 Merge branch 'test' into warmifx_customer_filters
jsegarra reviewed 2025-01-27 13:09:45 +00:00
src/pages/Customer/CustomerList.vue
@ -61,3 +65,4 @@
},
{
align: 'left',
jsegarra commented 2025-01-27 13:09:45 +00:00
Author
Member
Copy Link

Verás que hay muchas líneas iguales
Lo que he hecho es cambiar el orden, para que name sea el segundo

Verás que hay muchas líneas iguales Lo que he hecho es cambiar el orden, para que name sea el segundo
jsegarra added 1 commit 2025-01-27 13:31:02 +00:00
gitea/salix-front/pipeline/pr-test This commit looks good Details
b266c44d34 fix: sage params
jsegarra added 1 commit 2025-01-27 13:31:59 +00:00
gitea/salix-front/pipeline/pr-test This commit looks good Details
ba3f6a79e6 style: customerList
jsegarra requested review from alexm 2025-01-27 13:32:46 +00:00
alexm approved these changes 2025-01-27 13:37:10 +00:00
src/pages/Customer/CustomerList.vue Outdated
@ -438,4 +439,3 @@
emit-value
auto-load
>
<template #prepend>
alexm commented 2025-01-27 13:37:08 +00:00
Member
Copy Link

Si es intencional, fusionar

Si es intencional, fusionar
jsegarra commented 2025-01-27 18:05:40 +00:00
Author
Member
Copy Link

Es intencional y validado insitu por @jon

Es intencional y validado insitu por @jon
jsegarra commented 2025-01-28 06:11:20 +00:00
Author
Member
Copy Link

@alexm está Ok
Procedo a fusionar?

@alexm está Ok Procedo a fusionar?
jon commented 2025-01-28 06:11:49 +00:00
Member
Copy Link

Es correcto porque el componente ya tiene ese mismo template. También se creó una prop para mostrar el avatar

Es correcto porque el componente ya tiene ese mismo template. También se creó una prop para mostrar el avatar
jsegarra added 1 commit 2025-01-27 18:05:17 +00:00
gitea/salix-front/pipeline/pr-test This commit looks good Details
e923ebcc72 Merge branch 'test' into warmifx_customer_filters
jsegarra commented 2025-01-27 19:40:37 +00:00
Author
Member
Copy Link

Te adjunto report

Te adjunto report
pass_2025-01-27T185910+0100-report.zip
360 KiB
alexm commented 2025-01-28 06:17:05 +00:00
Member
Copy Link

Si fusionad

Si fusionad
alexm commented 2025-01-28 06:17:24 +00:00
Member
Copy Link

Fusiono y me hago pull para la subida

Fusiono y me hago pull para la subida
👍 1
alexm merged commit 8b916016c6 into test 2025-01-28 06:17:31 +00:00
alexm deleted branch warmifx_customer_filters 2025-01-28 06:17:31 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
alexm
Labels
Clear labels   
bug

Something is not working

  
CR / Tests passed

CR / Tests passed

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
invalid

Something is wrong

  
Missing Tests

Missing Tests

  
wontfix

This won't be fixed

No Label
bug
CR / Tests passed
duplicate
enhancement
invalid
Missing Tests
wontfix
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 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#1131
No description provided.
Delete Branch "warmifx_customer_filters"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?