PR-CUSTOMER #186
No reviewers
jsegarra
Labels
No Milestone
No Assignees
5 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix-front#186
Loading…
Reference in New Issue
No description provided.
Delete Branch ":PR-CUSTOMER"
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?
PR para el módulo de clientes.
https://redmine.verdnatura.es/issues/6823
@ -1,13 +1,12 @@<script setup>En el formulario de crear customer o basic-data, hemos encontrado que el campo email permite añadir cualquier formato. Propuesta, añadir type="email"
Este campo email, tampoco tiene el icono de info
Corregido:
3cf3687b84El formulario de crear customer no tiene el mismo formato que en Salix
Corregido:
17a0afda73@ -1,13 +1,12 @@<script setup>import { ref, computed, onBeforeMount } from 'vue';import { ref, computed } from 'vue';En customer/1112/summary, en el componente descriptor tenemos:
En esa misma ruta pero en la parte central, no se visualiza la tabla de "Últimos tickets"
Corregido:
17a0afda73El icono de información de riesgo, no lo veo añadido...es correcto?
Se agrega icono faltante:
d0cf13d0afRiesgo está repetido 2 veces. En uno está el icono y el otro no. quitar el que corresponda
@ -1,13 +1,12 @@<script setup>import { ref, computed, onBeforeMount } from 'vue';import { ref, computed } from 'vue';import { useI18n } from 'vue-i18n';En el formulario de basic-data:
El campo cliente anterior no aparece
Corregido:
3cf3687b84@ -1,3 +1,289 @@<script setup>Errores:
Corregido:
3cf3687b84Revisamos porque la columna referencia no tiene el mismo formato. El campo, debería ser descriptor, si procede
Corregido:
17a0afda73Revisar porque lago raro pasa
@wbuezas y yo hemos visto que las filas no siempre aparecen ordenadas con los customer 1102 y 1103.
He eliminado el "order:id DESC" y parece que va mejor, pero alguna vez aparece en otro orden
Pasaban 2 cosas:
Ambas cosas quedaron fixeadas en este commit:
1cff622898Aquí también hay un error en los datos que vienen del back porque el campo payed y created, tiene la misma fecha y hora para los registros con id 1
@ -3,3 +2,4 @@import { ref, computed } from 'vue';import { useI18n } from 'vue-i18n';import { QBtn, QCheckbox, useQuasar } from 'quasar';En fiscal-data, los campos: razón social, vies y recargo de equivalencia deberían tener el icono de info.
En billing-data, iban tampoco tiene.
Corregido:
11eb597fa3Carlos, en el comentario anterior indicaba sugerencias para fiscal-data y billing-data.
Billing-data, el campo IBAN sigue sin tener icono
No se hace ninguna corrección, se hace el commit:
6bf9b7c0c8solo deshaciendo los cambios que se hicieron en el commit11eb597fa3, quedando exactamente igual que antes del comentario.@ -4,4 +4,4 @@import { QBtn, QCheckbox, useQuasar } from 'quasar';import { toCurrency, toDate } from 'filters/index';Revisar los campos de los formularios de datos básicos, datos fiscales, formas de pago y consignatarios porque hay muchos campos que no tienen añadida la funcionalidad de clearable.
No pongo el nombre de cada campo de cada vista porque, como digo, son muchos.
Corregidos todos los de Customer:
7f1c4690e8Basic-data: comercialm canal de contacto, tipo de negocio. Son desplegables y otro campos similares si que tienen esa acción.
Habria que revisar porque en VnSelectAction/VnSelectCreate/VnSelectDialog lo añadimos pero no funciona, pero creo que esa pelota no está en vuestro tejado.
@ -0,0 +1,61 @@<script setup>En customer/1112/credits , el descriptor de la columna empleado, no está cogiendo el código del empleado sino del cliente.
Para el mismo cliente, en salix, está llamando a /api/Workers/100
Corregido:
11eb597fa3@ -0,0 +1,93 @@<script setup>El worker descriptor no va bien, porque usa clientFk en vez de worker, ya hay un comentario similar.
El campo/ valor de fecha no coincide...induce a error.
Las validaciones no se están aplicando
Corregido:
11eb597fa3Está OK.
Por mi parte queda comprobar si ha habido cambio de criterio en el WorkerDescriptorProxy
@ -0,0 +136,4 @@<QIcon name="close" size="sm" /></span><h5 class="flex justify-center q-mt-xs">Porqué está en el centro si el resto de dialogos si el resto está a un lateral?
Corregido:
3cf3687b84@ -0,0 +140,4 @@{{ t('New payment') }}</h5><VnRow class="row q-gutter-md q-mb-md">Los campos no son clearables
Corregido:
7f1c4690e8@ -0,0 +143,4 @@{{ t('Grade') }}:</div><div class="text-weight-bold">{{ item.insurances[0].grade }}Si no tiene valor aparece vacío. En Salix aparece un "-"
Corregido:
c0a9835e80@ -1,3 +0,0 @@<template>No identifico el nuevo componente, sin embargo, faltan clearables
Componente eliminado hace varios días
@ -1,3 +1,71 @@<script setup>Falta botón de cambiar contraseña
Los campos no son clearables
Corregido:
c0a9835e80Aparece el botón de cambiar contraseña, sin embargo, el formato del layout no es el estándar
Corregido:
ebd1ee07f4@ -0,0 +30,4 @@@on-data-saved="onDataSaved()"><template #form="{ data }"><VnRow class="row q-gutter-md q-mb-md">Los campos nos son clearables
Corregido:
7f1c4690e8@ -0,0 +76,4 @@<QCard class="q-pa-lg"><QCardSection><QForm @submit.prevent="onSubmit"><VnRowFalta clearable
Corregido:
7f1c4690e8@ -0,0 +89,4 @@<VnInput :label="t('Phone')" v-model="note.phone" /></div><div class="flex items-center">No funciona si has dado a guardar.
Puedes eliminar la primera pero si añades una fila y le das a guardar, reaparece.
Corregido:
d0e98010a5@ -0,0 +1,11 @@<script setup>Revisar comentario.
Si este componente, está así porque no hay datos, utilizar el cliente 1102
Corrección:
08e072100b@ -0,0 +1,240 @@<script setup>Revisar fecha de enviado y worker descriptor
En el create, revisar clearable
Corregido:
7f1c4690e8Aparece el botón de vista previa, sin embargo, el formato del layout no es el estándar
Corregido:
ebd1ee07f4@alexm @jgallego El funcionamiento del botón de restaurar, es correcto? cuando le pulsas, te elimina los campos del destinatario y responder a. Esta funcionalidad no está en Salix, por lo que quedaría por validar
Restaurar debe dejar el formulario como estaba cuando el usuario lo abrió, no se si con esto respondo a la pregunta.
@cfonseca Revisar restaurar en Consignees(customer/1101/adrress/create porque el campo Ubicación nose refresca.
Antes si que iba bien, pero trayéndote los cambios se ha quedado por validar esto.
Hay 4 ocurrencias de ese mismo componente, revisa las 4 y volvemos a revisar, ok?
Sigue sin ir del todo fino, adjunto video por el grupo.
Corregido:
5ff90d127bHay veces que los botones desaparecen, cambias de sección y sale error.
Siempre me sale en vista previa, el documento de Incoterms
Problema de error al cambiar sección y desaparición de botones resuelto.
Commit:
87f376464bSe fixea la vista previa del documento.
Commit:
ec2604b9ab@ -0,0 +116,4 @@url="Samples/visible"/><FormModelNo hay botón de vista previa
Corregido:
d0e98010a5Aparece el botón de vista previa, sin embargo, el formato del layout no es el estándar
Corregido:
ebd1ee07f4Revisar ya que el boton de vista previa está activo a pesar de no tener seleccionada la plantilla.
Si haces F5, el campo plantilla, por defecto no está seleccionada y el boton desactivado, eso está OK.
@ -0,0 +117,4 @@/><FormModel:form-initial-data="initialData"Cuando se muestra el campo dirección,. no muestra el nombre del consignatario sino otro valor
La label del campo debería ser consignatario
¿Debería tener el icono del lápiz para editar dirección? @alexm
Corrección:
08e072100b@ -0,0 +104,4 @@</div></VnRow><div class="flex justify-between q-mt-xl">Comportamiento anómalo, y muy similar a remove, combinando acciones de crear, guardar, eliminar.
Corregido:
d0e98010a5Me acaba de pasar que he creado una linea, he guardado y no me ha dejado volver a darle a guardar hasta hacer F5
Corregido:
bb10714824Revisar
@ -0,0 +230,4 @@</QPage><QPageSticky :offset="[18, 18]"><QBtnEl icono de fichero es diferente a otras partes de la aplicación. Unificar criterios. Puesto que este es el último módulo que quiere integrarse, lo reemplazaría por los que ya están.
Corregido:
3cf3687b84@ -0,0 +1,11 @@<script setup>Usar el cliente 1101
Corrección:
08e072100bLos iconos de la columna estado, al hacer hover cambia el cursor induciendo a error al usuario porque parece que se pueda hacer una acción.
El tooltip de icono de la ultima columna( mas a la derecha), no corresponde. en salix es "confirmar transaccion" y en la PR es Transacción confirmada"
Corregido:
bb10714824Revisar
@ -77,3 +72,1 @@</QIcon></template></VnInput><VnInput :label="t('IBAN')" clearable v-model="data.iban" />mmm...se ha eliminado el icono de info??
#186 (comment)
@ -0,0 +61,4 @@creates: notes.filter((element) => element.$isNew),};try {await axios.post('ClientContacts/crud', payload);No hay notificación de guardado?
Corregido:
ab20ec1738@ -0,0 +86,4 @@</div><div class="col"><VnInput :label="t('Phone')" clearable v-model="note.phone" />Te deja cualquier cosa
Corregido:
ab20ec1738@ -149,4 +140,1 @@{{ t('New consignee') }}</QTooltip></QPageSticky></template>error que no pertenece a esta linea, pero sí al fichero:
En la linea
<QIcon name="star" size="md" color="primary" />se muestra el icono de la estrella, debe estar marcado si el campos isDefaultAddress es <> false
se puede observar un ejemplo en http://localhost:5000/#!/client/1110/address/index
Corregido:
86199d8197@jgallego Yo lo he visto OK
@ -247,0 +269,4 @@<QTooltip>{{t("In order to invoice, this field is not contulted, but the consignee's ET. When modifiying this field if the invoice by address option is not checked, the change will be automatically propagated to all addresses, otherwise the user will be asked if he wants to propagate it or not"el modelo se llama address. Reemplazar todos los textos y rutas donde aparezca consignee y poner address
En salix esta así http://localhost:5000/#!/client/1101/address/create
Corregido:
ab20ec1738Yo lo he visto OK
@ -1,2 +14,3 @@<template><div class="flex justify-center">Web access</div><FormModeldesde salix http://localhost:9000/#/customer/1101/web-access cambio BruceWayne por BruceWayne4 y funciona, desde http://localhost:9000/#/customer/1101/web-access me da dos errores
id=1101' at line 1""UPDATE
account.userSET WHEREid=1101"Parece que en el set falta el name = xx
Corregido:
5e2c668fecYo lo he visto OK
11eb597fa3@ -216,2 +229,4 @@businessType: 'Tipo de negocio',contact: 'Contacto',youCanSaveMultipleEmails:'Puedeguardar varios correos electrónicos encadenandolos mediante comas sin espacios, ejemplo: user@dominio.com,user2@dominio.com siendo el primer correo electrónico el principal',Falta espacio en "puedeguardar"
Corregido:
bb10714824@ -200,3 +211,4 @@:rules="validate('bankEntity.countryFk')"hide-selectedoption-label="country"option-value="id"El campo Tipo de transaccion Stage, muestra mal los datos
Corregido:
86199d8197@ -286,9 +317,11 @@ es:Frozen: CongeladoHas to invoice: FacturaVies: ViesWhen activating it, do not enter the country code in the ID field: Al activarlo, no informar el código del país en el campo nif.Uff, en el limite. Si no es muy costoso (en tiempo), aplicar el mismo cambio que más abajo
Corregido:
86199d8197@ -289,3 +321,4 @@Notify by email: Notificar vía e-mailInvoice by address: Facturar por consignatarioIs equalizated: Recargo de equivalenciaIn order to invoice, this field is not contulted, but the consignee's ET. When modifiying this field if the invoice by address option is not checked, the change will be automatically propagated to all addresses, otherwise the user will be asked if he wants to propagate it or not: Para facturar no se consulta este campo, sino el RE de consignatario. Al modificar este campo si no esta marcada la casilla Facturar por consignatario, se propagará automaticamente el cambio a todos lo consignatarios, en caso contrario preguntará al usuario si quiere o no propagar.Reemplazar la key de la traduccion por una palabra camelCase, tipo "inOrderToInvoice".
El exto en inglés deberá estar en la etiqueta en
Corregido:
86199d8197@ -0,0 +33,4 @@@click.stop="setClientsConfirmTransaction"class="cursor-pointer"color="primary"name="check"Reemplazzar check por done_all que es el nombre del icono que está en Salix
Corregido:
86199d8197@ -3,0 +263,4 @@class="dotted-border-btn"no-caps/><QPopupEditRevisar, si este es el componente que acordamos en la UX/UI.
Hablar con @buezas porque ha hecho un cambio en la parte de travel/extra-community y el tipo de componente quasar que se está usando aquí no es el mismo
Corregido:
86199d8197@ -79,3 +56,3 @@label: t('Since'),name: 'created',format: (value) => date.formatDate(value, 'DD/MM/YYYY hh:mm:ss'),format: (value) => toDateHour(value),Revisar, porque en Salix, se muestra dd/mm/yyy HH:MM
Corregido:
86199d8197Duda, cuando actualicemos con dev no tendremos conflicto?
PR-CUSTOMERto WIP: PR-CUSTOMERWIP: PR-CUSTOMERto PR-CUSTOMER@ -0,0 +103,4 @@@click="toCustomerAddressEdit(item.id)"><div class="q-ml-xs q-mr-md flex items-center"><QIconLos consignatarios se ordenan poniendo el predeterminado (star filled) el primero y el resto despues, mirar orden en salix.
La estrella tiene un evento click que sirve para marcar como predeterminado otro de los consignatarios, mirar funcionamiento en salix.
Corregido:
980904a382@cfonseca El funcionamiento que reportaba Javi, está OK.
Sin embargo, falta revisar el layout de address/:id/edit:
Ver imagen de chat
Corregido:
7eca27b8afMmm...falta revisar VnLocation porque no está cargando el valor.
Si quieres nos juntamos en post-daily y lo vemos.
He hecho la prueba y se actualiza el campo correctamente. También carga el valor correctamente
@ -74,3 +72,4 @@<VnInput :label="t('IBAN')" clearable v-model="data.iban"><template #append><QIcon name="info" class="cursor-info"><QTooltip>{{ t('components.iban_tooltip') }}</QTooltip>componente CreateBankEntityForm despues de crear un bankEntity lo debe insertar en el textField tal como hace salix
@cfonseca He visto modificado este componente en otro commit, revisa por favor y si no está abordado, cógelo. gracias
Corregido:
b12968f982La funcionalidad está OK pero adjunto imagen porque al cargar el formulario esta validado y en vencimiento hay un 0 pero lo pone en rojo
Si limpio el campo y le doy a restaurar, se vuelve a poner en rojo
Corregido:
5ff90d127b@ -3,0 +78,4 @@data-key="CustomerCreditContracts"order="id DESC"ref="vnPaginateRef"url="CreditClassifications"cuando en la tabla creditClassification no hay dateEnd no muestra resultados, en salix sí lo hace. Además en este escenario es donde aparece en salix un candado para cerrar el contrato,
ejectura este update, ir a salix y copiar el funcionamiento en lilium
UPDATE vn.creditClassification
SET dateEnd=NULL
WHERE id=5
Después de la prueba que hicimos en el daily todo funcionó correctamente, quedo pendiente por si debo hacer alguna corrección
@ -3,0 +57,4 @@v-model.number="data.rating"/></div><div class="col">no inserta, en salix añado una clasificacion ej: 234 sin credito recomendado, lo inserta y lo muestra bajo,
en lilium da un error y no lo muestra, aun teniendo datos en la tabla insertados via salix no son mostrados en lilium
Hice la prueba y me muestra este error: message: "La razón social debe ir en mayúscula", fui a la pantalla de Datos fiscales y modifique el campo de Razón social y Dirección fiscal y ya con estos cambios se pudo guardar el valor de Clasificación en la pantalla de Opinión de crédito sin problemas, por favor revisar, quedo pendiente
@cfonseca Esto es lo que vimos en la daily antes de Luciano no? Puedes confirmar? Gracias
@ -46,17 +46,24 @@ function handleLocation(data, location) {:label="t('Social name')"Un poco mas arriba en esta linea algo debe haber mal porque si intento elegir un Tipo de transaccion Sage me escribe [object object}, con tipo de impuesto va bien.
url="SageTransactionTypes"
Corregido:
3bcd4984f5Lo veo OK
@ -279,1 +280,4 @@</QCard><QCard><div class="header">{{ t('Latest tickets') }}falta traduccion
Corregido:
3bcd4984f5Lo veo OK
@ -0,0 +39,4 @@<VnRow class="row q-gutter-md q-mb-md" v-if="unpaidClient"><div class="col"><VnInputDate :label="t('Date')" v-model="data.dated" />salix pone una fecha por defecto, emular comportamiento
Corregido:
90ee50eab5Lo veo OK
@ -43,3 +150,3 @@</div><div class="col"><VnInput :label="t('Recovery email')" v-model="item.email"><VnInputubicar el mail bajo del usuario como en salix, solo hay 3 campos la lectura es mas facil
Corregido:
3bcd4984f5Lo veo OK
@ -0,0 +96,4 @@const payload = {creates: notes.value,};await axios.post('AddressObservations/crud', payload);http://localhost:9000/#/customer/6/address/259/edit
se añade una primera observacion en itemPicker funciona bien, al añadir una segunda en Delivery dice que no se puede repetir. Esto mismo en salix sí funciona.
Corregido:
2c70a73b8eYo he probado a guardar Packager, delivery e ItemPicker y ha guardado resultados.
Yo lo veo OK
@ -0,0 +112,4 @@try {const { data } = await axios.get(url);quasar.dialog({abrir como en salix que es mas ancho, mas similar a un dina4 que es lo que intuitivamente espera el usuario
Corregido:
90ee50eab5Yo lo veo OK
@ -401,3 +430,4 @@meta: {title: 'others',icon: 'pending',menuChildren: [en salix al abrir este menu el scroll baja para poder ver directamente las opciones desplegadas, copiar funcionamiento.
Corregido:
c767bcde7d@ -404,0 +439,4 @@{name: 'CustomerConsumption',title: 'consumption',icon: 'vn:lines',salix tiene otro icono
Corregido:
3bcd4984f5Yo lo veo OK
@ -404,0 +464,4 @@{name: 'CustomerUnpaid',title: 'unpaid',icon: 'vn:supplierfalse',salix tiene otro icono
Corregido:
9ae0874ccfLo veo OK
@ -406,0 +538,4 @@title: 'webPayment',},component: () =>import('src/pages/Customer/Card/CustomerWebPayment.vue'),si estoy en el cliente 1110 que no tiene pagos web, y en el buscador superior me muevo al 1101 que sí los tiene en salix me aparecen en lilium tengo que hacer f5 para que acceda a la información, no pasa en esta seccion solamente, tal vez alguien de vuestro o nuestro equipo esto ya lo haya solucionado en algun otro formulario..preguntar a ver.
Corregido:
b12968f982@jgallego Tenemos un comportamiento similar en el resto de la aplicación.
Ya tenemos un redmine abierto #186 (comment)
La solución de Carlos es similar a la que se ha aplicado en el redmine
@ -406,0 +592,4 @@title: 'unpaid',},component: () =>import('src/pages/Customer/Card/CustomerUnpaid.vue'),esta seccion da fallo simplemente al entrar
Corregido:
3bcd4984f5@jgallego Este método también falla en Salix, lo que pasa es que no se muestra mensaje de error
URL=> http://localhost:5000/api/ClientUnpaids/1112?filter=%7B%7D
Lo arregla verdnatura
Negativo!
@cfonseca, la manera correcta de hacer uso de esta tabla no es mediante el path "/ClientUnpaid/:id", sino pasándole el clientFk como filter
Ten en cuenta que tendrás que añadir registros a la tabla vn.clientUnpaid. bien lo puedes hacer desde la extensión de vscode que comentamos, desde DBeaver o desde fixtures.before.sql...si ves que no, apúntatelo para verlo en la daily
Corregido:
e56aa6587fHe añadido un comentario a este código.
Tras llamada, Carlos va a hacer unos cambios ya que la implementación estaba bien orientada pero la ejecución no era la correcta,
@ -0,0 +19,4 @@const state = useState();const user = state.getUser();const initialFormState = reactive({Estamos viendo en este módulo y otros que da errores al llamar a salix
Lo que pasa es que si solo modificas un campo el resto se envía estando igualmente a null, este planteamiento es correcto?
Corregido:
efb676e4a4Not fixed!
Me muestra el mensaje de "Desea salir sin guardar", pero se cierra solo y la operación POST se resuelve con éxito
Si, aparentemente quedo un problema en el
FormModelen el manejo debeforeRouteLeavecuando lo implementaron, quedó solucionado.Commit:
5ae54d557b@ -102,3 +102,3 @@<template #body="{ entity }"><VnLv :label="t('worker.card.name')" :value="entity.user?.nickname" /><VnLv :label="t('worker.card.email')" :value="entity.user?.email" copy /><VnLv :label="t('worker.card.emailxxx')" :value="entity.user?.email" copy />🚩
Corregido:
b12968f982@ -0,0 +61,4 @@<template><Teleport to="#st-actions"><QBtnGroup push class="q-gutter-x-sm">No se muestran los botones
Los botones siempre estaban visibles, corregí que cuando habían datos no se estaban mostrando adecuadamente y no recargaba la data al cambiar manualmente la url:
7eca27b8af@ -90,3 +125,4 @@><QTooltip>{{ t('customer.card.notChecked') }}</QTooltip></QIcon><QBtnRevisar tooltip porque no está igual.
Lo del icono va a parte
Corregido:
0a33b8d92aPara probar hay que añadir valores a la fixtures.before.sql
@ -0,0 +48,4 @@const { data } = await axios.get('ClientUnpaids', {params: { filter: JSON.stringify(filter) },});const unpaid = data.find((element) => element.clientFk == id);Porque haces un find por id, si ya lo has hecho en la query/filter? Te devoilverá el resultado
Si filtras por 1101, te llegará [1101]
Corregido:
93015d9f98@ -0,0 +49,4 @@params: { filter: JSON.stringify(filter) },});const unpaid = data.find((element) => element.clientFk == id);if (unpaid) {Mi propuesta es si data.length>0
Corregido:
93015d9f98@ -94,34 +99,34 @@ const columns = computed(() => [{align: 'left',todos los campos de euros deben de ir alineados a la derecha como en salix.
Así se ve para el cliente 1101. Ver imagen adjunta
Sin embargo, el cursor cambia a "la mano", induciendo a que se puede cambiar, cuando debería aparecer el prohibido o cambiar el color del checkbox a gris
Corregido, campos de tipo moneda alineados a la derecha y el checkbox deshabilitado:
0aa5a051fbYo lo he visto OK
El checkbox es de color naranja pero se ve que está deshabilitado y tiene el cursor en prohibido
@ -107,4 +112,4 @@},{align: 'left',field: 'userName',El usuario en minusculas igual que salix (cambiar para todos los sitios que se muestre)
Corregido:
663010a0d4Yo lo he visto OK
@ -125,3 +129,4 @@name: 'bank',},{align: 'left',el cambo debit son euros le falta
format: (value) => toCurrency(value),
Corregido:
f8b698e4e7Yo lo he visto OK en customer/1101/balance
@ -220,1 +270,4 @@:id="props.row.workerFk"v-if="props.col.name === 'employee'"/></component>http://localhost:5000/#!/client/100/balance/index en salix hay un registro y el boton de enviar compensación, en lilium no aparece ni el registro ni el boton
Corregido:
f8b698e4e7hace falta cambiar el icono del mail, no lo he encontrado en Lilium aunEfectivamente no está.
Hablamos con el responsable de la iconografía
Gracias
Resuelto
@ -277,3 +342,4 @@bill: 'N/INV {ref}'Bank: CajaDebit: DebeHavings: Haberesta seccion va con scroll infinito como en salix
Corregido:
09a2af4124@ -126,3 +136,4 @@{align: 'left',field: 'debit',format: (value) => toCurrency(value),en las columnas debe y haber si el valor es cero no mostrarlo
tal como lo hace en salix.
Corregido:
d74119f433Yo lo he visto OK
@ -138,3 +148,4 @@name: 'havings',},{align: 'left',Revisar en salix el valor del campo balance
http://localhost:5000/#!/client/1101/balance/index
respecto a lilium
http://localhost:9000/#/customer/1101/balance
el balance es un saldo por tanto, va acumulando el balance anterior +- el valor del debe/haber
Corregido:
d74119f433Not fixed! Ver imagen
Corregido el valor del campo balance:
38d3d49ecbHe probado con los clientes 1101, 1102 y 1103 y lo he visto OK
@ -148,3 +159,3 @@field: 'isConciliate',label: t('Conciliated'),name: 'isConciliate',name: 'conciliated',no se porque el conciliado al lado del checkbox tiene un 1, http://localhost:9000/#/customer/1101/balance
Corregido:
d74119f433Yo lo he visto OK
@ -40,10 +34,10 @@ const tableColumnComponents = {'model-value': Boolean(row.selected),http://localhost:9000/#/customer/1101/balance campo isWorker aunque en la BBDD este marcado no se marca el checkBox.
Corregido:
c844d26ccbYo lo he visto OK
@ -41,3 +35,3 @@}),},salesperson: {salesPerson: {En salix las columnas tienen un tooltip y son ordenables
ademas se tiene que ajustar al ancho de la pantalla sin scroll lateral.
Corregido:
5ff90d127bYo lo he visto OK, en cuanto al numero de columnas con y sin tooltip
@ -0,0 +25,4 @@<FormModel:form-initial-data="initialData":observe-form-changes="false":default-actions="true"Por defecto, el valor es true, así que creo que no es necesario definirla
Corregido:
38d3d49ecb@ -27,3 +23,3 @@<template><FormModel:form-initial-data="initialData":form-initial-data="{}"Diria que esta tampoco hace falta, de echo, solo hay 3 ocurrencias y corresponden a esta PR
Corregido:
38d3d49ecb@ -0,0 +89,4 @@</template><i18n>es:TIP: quizás usando traducciones con argumentos nos podemos ahorrar 2 líneas
Corregido:
1e9b3f8f0e@ -0,0 +59,4 @@const formData = new FormData();if (dms.value.files) {for (let i = 0; i < dms.value.files.length; i++)Comentario, no es nuestro estilo, hay otras instrucciones para hacer lo mismo pero con menos código.
Aunque no es su estilo, la repetición de código en varios archivos me intrigó. Viendo espacio para mejora, investigué su origen y hallé varias instancias. Curioso para ser 'no característico' de su enfoque, ¿no?
Corregido:
38d3d49ecbFantástico Carlos, tomaré tu commit como ejemplo para nuevos casos y cuando surja la oportunidad cambiar las otras ocurrencias!
@ -0,0 +1,3 @@[ZoneTransfer]Eliminar fichero
Corregido:
38d3d49ecb@ -0,0 +1,3 @@[ZoneTransfer]Eliminar fichero
Corregido:
38d3d49ecb@ -166,4 +175,4 @@securedCredit: 'Crédito asegurado',payMethod: 'Método de pago',debt: 'Riesgo',isDisabled: 'El cliente está desactivado',Comentario, ¿Podemos cambiar "El cliente {verbo} {estado}" por "Cliente {estado}"?
Cliente inactivo
Cliente congelado
Cliente con riesgo
cliente no comprobado
Así está en Salix:
Client frozen: Cliente congelado
Client has debt: Cliente con riesgo
Client inactive: Cliente inactivo
Client not checked: Cliente no comprobado
Gracias
Los elementos a los cuales este comentario hace referencia fueron creados hace 17 MESES, igualmente lo corregí.
Corregido:
38d3d49ecbGenial!
Gracias Carlos
@ -0,0 +102,4 @@/></div></VnRow>El layout debe tener(salto de linea = 1 fila)
1 checkbox
2 inputs
El componente VnLocation ocupa toda la fila
3 inputs
1 inputs y un input con el botón de crear
Corregido:
38d3d49ecb@ -0,0 +1,73 @@<script setup>Se que no es este archivo, pero está relacionado.
El candado de "cerrar contrato" no está en línea con el texto y le falta el relleno.
Ver adjunto
Corregido:
38d3d49ecb@ -1,17 +1,17 @@version: '3.7'Esto no debería estar
@ -0,0 +1,3 @@[ZoneTransfer]eliminar fichero
Corregido:
38d3d49ecb@ -0,0 +1,3 @@[ZoneTransfer]eliminar fichero
Corregido:
38d3d49ecb@ -0,0 +1,3 @@[ZoneTransfer]eliminar fichero
Corregido:
38d3d49ecb@ -0,0 +1,3 @@[ZoneTransfer]eliminar fichero
Corregido:
38d3d49ecb@ -0,0 +1,3 @@[ZoneTransfer]eliminar fichero
Corregido:
38d3d49ecb@ -22,3 +22,3 @@template: {type: String,required: true,required: false,Y si ponemos comillas simples como default reduciríamos la complejidad de usar ternario,no?
Sale un warning "Prop 'template' requires default value to be set." Lo digo por poner las comillas y nos evitamos un ternario
Apliqué un string vacío
''como default de la proptemplate, pero sinceramente no se me ocurre como mejorar el ternary operator ese.Pensé algo como:
const key = props.template ?templates['${props.template}']: props.template;.Pero no se si tiene mucho sentido la verdad.
Commit:
bafb938e23@ -0,0 +1,11 @@export default function toDateHourMin(date) {Duda, cuando actualicemos con dev seguirá siendo útil?
En el archivo
date.jsestán las nuevas utils que hacen esto así que supongo que ya no sería útilCustomerBalance CustomerCreditOpinion y CustomerCredit usan toDateHourMin
@ -0,0 +1,12 @@export default function toDateHourMinSec(date) {Duda, cuando actualicemos con dev seguirá siendo útil?
En el archivo
date.jsestán las nuevas utils que hacen esto así que supongo que ya no sería útilTodavia no, porque hay otras ocurrencias que lo usan, CustomerWebPayment y CustomerCreditContractsInsurance
Lo anoto para eliminar en nuestro lado VnNotes y OrderSummary
@ -0,0 +163,4 @@field: (value) => value.dms.created,label: t('Created'),name: 'created',format: (value) => toDateHourMinSec(value),Duda, cuando actualicemos con dev no tendremos conflicto?
Se reemplazo
toDateHourMinSecpor las nuevas utils ubicadas endate.jsCommit:
3cb045936b@ -92,3 +74,3 @@label: t('Date'),name: 'date',format: (value) => date.formatDate(value, 'DD/MM/YYYY hh:mm:ss'),format: (value) => toDateHourMinSec(value),Duda, cuando actualicemos con dev no tendremos conflicto?
Se reemplazo
toDateHourMinSecpor las nuevas utils ubicadas endate.jsCommit:
73ea49df63@ -0,0 +74,4 @@field: 'created',label: t('Register date'),name: 'registerDate',format: (value) => toDateHourMinSec(value),Duda, cuando actualicemos con dev no tendremos conflicto?
Se reemplazo
toDateHourMinSecpor las nuevas utils ubicadas endate.jsCommit:
093c367d7e@ -56,0 +27,4 @@<QCard class="card-width q-pa-lg"><VnPaginatedata-key="CustomerNotes":url="'clientObservations'"Si es texto literal , quitar :
Aplicado
Commit:
fb59ac1f2f@ -56,0 +37,4 @@v-for="(item, index) in rows":key="index":class="{'q-pa-md': true,TODO: si lleva true mover a class
Se mejoró la definición de clases
Commit:
f6b8a73ae2@ -56,0 +49,4 @@</p><p class="color-vn-label">{{date.formatDate(Duda, cuando actualicemos con dev no tendremos conflicto?
Se reemplazo
date.formatDatepor las nuevas utils ubicadas endate.jsCommit:
e1f8b7217a@ -0,0 +55,4 @@field: 'created',label: t('Sent'),name: 'sent',format: (value) => toDateHourMinSec(value),Duda, cuando actualicemos con dev no tendremos conflicto?
Se reemplazo
toDateHourMinSecpor las nuevas utils ubicadas endate.jsCommit:
c856adc8d0@ -0,0 +15,4 @@const { notify } = useNotify();const stateStore = useStateStore();const initialDated = '2001-01-01T11:00:00.000Z';TODO, revisar si hay otra opción
Modificado
Commit:
998561badd@ -0,0 +136,4 @@<VnInputDate :label="t('Date')" v-model="dated" /></div><div class="col"><VnInputautofocus. Aunque no lo apliques porque tenemos un desarrollo para evitar esta propiedad
@jsegarra autofocus agregado!
Commit:
82f259be32@ -85,6 +85,7 @@ defineExpose({const componentIsRendered = ref(false);onMounted(async () => {originalData.value = $props.formInitialData;Eliminar el import de VnRow
Mmmm... No veo un import de
VnRowen el componenteFormModel@ -53,3 +53,4 @@const key = props.template ? `templates['${props.template}']` : '';message.value = t(key, params, { locale: locale.value });}El estilo del modal no es el mismo
El campo subject no se está enviando en el body del POST, por lo que lo quitaria. En Salix, ese campo no existe
En mensaje falta añadir el icono de info
Se adapto al estilo de los inputs de salix y se agregó el tooltip.
@jgallego no quise tocar es el input de
Subjectque @jsegarra menciona porque lo agregó un chico de verdnatura y supongo que habrá sido por algo, prefiero de momento no tocarlo al menos que sea estrictamente necesario.Commit:
94644de4faok, lo dejamos así
@ -0,0 +1,253 @@<script setup>import { onBeforeMount, ref, watch } from 'vue';Se que no aplica ni es este fichero, pero podemos eliminar
const route de CustomerCard??
Sale warning de consola
Route eliminado
Commit:
3e124581c4@ -35,6 +33,16 @@ const toCustomerNotes = () => {@on-data-saved="toCustomerNotes()"url-create="ClientObservations"><template #moreActions>@alexm VnNotes??
@ -0,0 +6,4 @@const { t } = useI18n();const $props = defineProps({Eliminar porque sale warning
Asignación de prop eliminada:
Commit:
107eaea704New commits pushed, approval review dismissed automatically according to repository settings