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

warmifx: Customer_filters #1131

Open
jsegarra wants to merge 2 commits from warmifx_customer_filters into test
pull from: warmifx_customer_filters
merge into: verdnatura:test
Conversation 0 Commits 2 Files Changed 1 +35 -12
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

@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
jsegarra added 2 commits 2024-12-24 11:17:05 +00:00
gitea/salix-front/pipeline/pr-test This commit looks good 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
@ -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
All checks were successful
gitea/salix-front/pipeline/pr-test This commit looks good
Required
Details
This pull request has changes conflicting with the target branch.
  • src/pages/Customer/CustomerList.vue
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?