warmifx: Customer_filters #1131

Merged
alexm merged 10 commits from warmifx_customer_filters into test 2025-01-28 06:17:31 +00:00
1 changed files with 0 additions and 22 deletions
Showing only changes of commit 9dd216269a - Show all commits

View File

@ -436,32 +436,10 @@ function handleLocation(data, location) {
<VnSelectWorker <VnSelectWorker
:label="t('customer.summary.salesPerson')" :label="t('customer.summary.salesPerson')"
v-model="data.salesPersonFk" v-model="data.salesPersonFk"
:params="{
departmentCodes: ['VT', 'shopping'],
}"
Outdated
Review

Si es intencional, fusionar

Si es intencional, fusionar
Review

Es intencional y validado insitu por @jon

Es intencional y validado insitu por @jon
Review

@alexm está Ok
Procedo a fusionar?

@alexm está Ok Procedo a fusionar?
Review

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
:has-avatar="true" :has-avatar="true"
:id-value="data.salesPersonFk"
emit-value emit-value
auto-load auto-load
> >
<template #prepend>
<VnAvatar
:worker-id="data.salesPersonFk"
color="primary"
:title="title"
/>
</template>
<template #option="scope">
<QItem v-bind="scope.itemProps">
<QItemSection>
<QItemLabel>{{ scope.opt?.name }}</QItemLabel>
<QItemLabel caption
>{{ scope.opt?.nickname }},
{{ scope.opt?.code }}</QItemLabel
>
</QItemSection>
</QItem>
</template>
</VnSelectWorker> </VnSelectWorker>
<VnLocation <VnLocation
:acls="[{ model: 'Province', props: '*', accessType: 'WRITE' }]" :acls="[{ model: 'Province', props: '*', accessType: 'WRITE' }]"
Outdated
Review

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

Cierto, hice, literal, copy paste

Cierto, hice, literal, copy paste

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

@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
Outdated
Review

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í

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

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

resuelto

resuelto