CambiosSolicitadosSuppliers #214
No reviewers
jgallego
Labels
No Milestone
No Assignees
5 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix-front#214
Loading…
Reference in New Issue
No description provided.
Delete Branch ":CambiosSolicitadosSuppliers"
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?
https://redmine.verdnatura.es/issues/6540
Esta PR responde los siguientes cambios solicitados:
1- De inicio que no aparezca ningun registro como en salix
http://localhost:9000/#/supplier/791/summary los links tienen que ir a lilium
2- El descriptor de supplier sólo muestra el icono de ir al cliente cuando hay un cliente posible
3- http://localhost:9000/#/supplier/791/basic-data al guardar activo se debe reflejar en el descriptor :marca_de_verificación_blanca:
4- http://localhost:5000/#!/supplier/791/account entidad bancaria salix muestra BBKKESMMMMM en lilium muestra el banco
5- http://localhost:9000/#/supplier/791/contact al hacer click en el mas el foco debe ir al primer campo "Nombre"
6- http://localhost:9000/#/supplier/791/address/create usar el mismo componente del codigo postal que lo incluye todo igual que la seccion http://localhost:9000/#/supplier/791/fiscal-data
7- http://localhost:9000/#/supplier/791/fiscal-data si añado un codigo postal lo tiene que cambiar por el actual
Aclaración:
Con @jsegarra quedamos en que los puntos 6 y 7 quedaban en manos de el equipo de Verdnatura
@ -54,2 +54,3 @@<template #header-left><a v-if="isAdministrative" class="header link" :href="supplierUrl"><router-linkv-if="isAdministrative && route.name !== 'SupplierSummary'"Duda, pero esto se tiene que hacer en cada módulo?
No se puede hacer a nivel de CardSummary?
@jsegarra actualmente eso se maneja de manera externa al
CardSummary, lo que si se puede hacer es integrar ese icon dentro delCardSummaryy no repetirlo en todas las vistasAcá esta el cambio del
CardSummarydel que hablamosCommit:
f1961f07d1Hola!
Una duda, el punto 5, se refiere a aquí?
Creo que no funciona el focus
Buenas @alexm!
No, el punto 5 se refiere supplier contacts: http://localhost:9000/#/supplier/791/contact
Te adjunto una captura de la vista
@ -114,7 +114,7 @@ onMounted(() => {:label="t('worker.create.bankEntity')"http://localhost:5000/#!/supplier/791/account entidad bancaria salix muestra BBKKESMMMMM en lilium muestra el banco, hay que hacer como en salix
al guardar y entrar de nuevo, sin desplegar debe mostrar el bic, no el nombre el banco como hace ahora
Corregido
Commit:
ff94d7e43d@ -71,6 +73,8 @@ const setData = (entity) => {data.value = useCardDescription(entity.ref, entity.id);};const currentSupplier = computed(() => state.get('supplier'));si cogemos el patron de usar entity igual que parece que hacia salix para guadar la entidad en la que nos encontramos mejor, @jsegarra confirma tu.
@wbuezas A lo que se refiere Javi es al cambio del nombre de la variable? Hay algún motivo explicito?
La verdad es que si ya estas en el descriptor sabes que es el currentSupplier. Quizás si fuese un formulario en el que manejas diferentes estados puedo entenderlo.
Por lo que he visto, al declarar como entity puede dar problemas de reactividad con los iconos
Aunque vemos que no se está usando este patrón en ClaimDescriptor, por ello, podemos cambiar currentsupplier por supplier?
Claro, tomé la decisión de cambiar el nombre de la variable porque en el html ya utilizamos la variable
entityque se obtiene delv-sloty en este caso necesitaba obtener la información del store de manera que se podian repetir/pisar las variables con el mismo nombreDe todas maneras cambié el nombre de la variable
currentSupplierpor el nombresupplierCommit:
052123ad0e@ -53,6 +53,7 @@ function handleLocation(data, location) {:url-update="`Suppliers/${route.params.id}/updateFiscalData`"model="supplier"auto-load:clear-store-on-unmount="false"http://localhost:9000/#/supplier/791/fiscal-data si añado un codigo postal desde el boton mas, lo tiene que cambiar por el actual
@jgallego habíamos charlado este asunto con @jsegarra y quedamos en que el equipo de Verdnatura se iba a encargar de los inputs relacionados a los
postcodes#214 (comment)
@jgallego https://redmine.verdnatura.es/issues/6973?issue_count=27&issue_position=1&next_issue_id=6972
Perfecto, cierro convesación
New commits pushed, approval review dismissed automatically according to repository settings
New commits pushed, approval review dismissed automatically according to repository settings
New commits pushed, approval review dismissed automatically according to repository settings
No funciona el focus
Aunque en salix pone foco en SWIFT, @wbuezas podemos poner la propiedad autofocus en CreateBankentityForm:63?
New commits pushed, approval review dismissed automatically according to repository settings
Listo @jsegarra, agregué el focus en el
Input SWIFTcuando se monta el formulario.Commit:
912d54ea55Yo la veo bien, pero teniendo en cuenta que esta ventana carga los resultados cuando pulsas la tecla enter en la searchbar, @wbuezas crees que podríamos hacer que al pulsar sobre el icono de la lupa en la searchbar, cargue resultados? Es un detalle pero estoy seguro que los usuarios a la larga nos reclamarían eso.
Puedes apoyarte en salix, que si lo tiene implementado
🔝🔝
@jsegarra claro, no hay problema!
Te adjunto el commit:
800b626c99New commits pushed, approval review dismissed automatically according to repository settings
New commits pushed, approval review dismissed automatically according to repository settings
New commits pushed, approval review dismissed automatically according to repository settings
New commits pushed, approval review dismissed automatically according to repository settings
New commits pushed, approval review dismissed automatically according to repository settings