Modulo de rutas #195
No reviewers
jgallego
Labels
No Milestone
No Assignees
4 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix-front#195
Loading…
Reference in New Issue
No description provided.
Delete Branch ":feature/route-module"
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?
No incluye la sección dms de autónomos no esta incluida.
https://redmine.verdnatura.es/issues/6546
Se ha detectado lo siguiente en el formulario de crear:
Se ha detectado lo siguiente en el summary de una ruta/6
Se ha detectado lo siguiente en route/6/tickets :
Se ha detectado lo siguiente en Autonomos
-Se ha detectado lo siguiente en Troncales
@ -90,3 +90,3 @@url="AgencyModes/isActive":filter="{ fields: ['id', 'name'] }"sort-by="name ASC"sort-by="name"En este archivo, si cambias el id del registro por la URL, se refresca el descriptor lateral pero no los datos
Te dejo un commit donde lo he solucionado para este tipo de layout
1fecb6b954Si veis una mejor manera de corregirlo, me comentas.
Corregido
53f69738a5.Ha dejado de funcionar la solucion porque pruebo a cambiarme entre el 3 y 7, pero de todas maneras hemos hecho nosotros una solucion en VnPaginate para corregir esto
@kmartinez El punto 5 no es error nuestro sino de la migración. Lo vemos cuando tengas un hueco
@ -229,3 +244,4 @@<QPage class="column items-center q-mb-xl"><VnSubToolbar class="bg-vn-dark justify-end"><template #st-actions><QBtnSe que no está en Salix, pero podríamos añadir un icono(eye) para que al pulsarlo dirija a route/:id/summary
Ahora hay que hacer 2 clicks
@ -131,7 +131,7 @@ const openBuscaman = async (route, ticket) => {</script>Mmm..hemos creado un Buscaman en utils pero estamos manteniendo la misma funcionalidad en dos sitios?
Corregido
d816a57272.@kmartinez El punto 5 no es error nuestro sino de migración. Lo vemos después e los otros comentarios.
No confundir este punto 5 con el otro punto 5
Este commit
698a88e6f0ayuda a manejar el infinite scroll en el componente VnPaginate para evitar ciclos infinitos.@ -0,0 +91,4 @@</span></template></VnLv><VnLv :label="t('ETD')" :value="toDateHour(entity?.etd)" />Segundos?
Se resuelve en una PR aparte
@ -0,0 +86,4 @@<VnInputDate v-model="roadmapStopForm.eta" :label="t('ETA date')" /></div><div class="col"><VnInputTime v-model="roadmapStopForm.eta" :label="t('ETA date')" />Es inputTime pero la label apunta a date
Corregido
3c68f4779f.@ -0,0 +41,4 @@><template #body="{ entity }"><VnLv :label="t('Roadmap')" :value="entity?.name" /><VnLv :label="t('ETD')" :value="toDateHour(entity?.etd)" />Segundos?
Se resuelve en una PR aparte
@ -0,0 +44,4 @@{name: 'ETA',label: t('ETA'),field: (row) => toDateHour(row?.eta),segundos?
Se resuelve en una PR aparte
@ -278,199 +276,136 @@ const markAsServed = () => {:rows-per-page-options="[0]"hide-pagination:pagination="{ sortBy: 'ID', descending: true }":no-data-label="t('No results')"❌ Revisamos porque not working para español
@ -39,3 +40,4 @@align: 'center',},{name: 'worker',Creo que no estamos mostrando correctamente el valor que toca
Corregido
e8e2bc8403.@ -104,3 +112,3 @@/><FormModel:url="isNew ? null : `Routes/${shelvingId}`":url="isNew ? null : `Routes/${route.params?.id}`"Agencia y creado están al revés, pero bueno, creo que no es relevante por eso lo pongo como comentario
@ -442,1 +373,3 @@</QPopupEdit><template #body-cell-started="{ row }"><QTd class="table-input-cell"><VnInputTimeRevisamos porque me añade una hora a la que he seleccionado
Si, selecciono 13:00 me aparece 14:00
Este error tiene que ver con el formato de las fechas.
Se soluciona en otro PR #223
@ -443,0 +373,4 @@<template #body-cell-started="{ row }"><QTd class="table-input-cell"><VnInputTimev-model="row.started"Si selecciono 12:00 o 00:00, que es lo mismo que ya aparece me dice invalid Date
Este error tiene que ver con el formato de las fechas.
Se soluciona en otro PR #223
@ -0,0 +285,4 @@</QTooltip></QBtn><QBtnicon="sms"añadir: font-variation-settings: 'FILL' 1;
Corregido
7974725da0.@ -0,0 +66,4 @@><div class="col"><VnSelectFilterv-model.number="row.warehouseFk"Hecho en falta el autofocus cuando es nuevo registro
Corregido
7974725da0.Uff...un ejemplo de, mala practica, tener código duplicado. Has añadido autofocus 1 vez pero ese formulario lo tenemos duplicado.
Lo del formulario duplicado lo comentamos en post-daily, pero entiendo que no te ha dado tiempo a todo
Aca se crea un nuevo componente para el formulario
1ab9bfefed.@ -0,0 +82,4 @@<VnInputDate v-model="row.eta" :label="t('ETA date')" /></div><div class="col"><VnInputTime v-model="row.eta" :label="t('ETA date')" />Tenemos definida una traducción para ETA hour pero no la estamos usando
Corregido
7974725da0.@ -0,0 +264,4 @@<QTooltip>{{ t('Open buscaman') }}</QTooltip></QBtn><QBtnicon="filter_alt"font-variation-settings: 'FILL' 1;
Corregido
7974725da0.@ -0,0 +60,4 @@});const onSubmit = async () => {if (!props.destination) {Teniendo arriba:
Hace falta estos ifs?
Mmm...es por si se modifica a posteriori. De todas maneras si borras el campo sin el clearable, te deja el valor del campo a "", por lo que la validación no entraría, no?
Quizás falta integrar lo del hover_clearable o cambiar la validación.
@kevin, tus has podido mostrar el throw de la validación?
Gracias
@ -0,0 +43,4 @@},{name: 'population',label: t('Population'),en salix es city.
"Population" se refiere al número total de individuos que habitan en un área geográfica específica
En Salix hay una incongruencia, en el listado de tickets se muestra "city", pero en el modal muestra "population".
Se cambio a "City".
e4553659cf@ -0,0 +10,4 @@import RouteListTicketsDialog from 'pages/Route/Card/RouteListTicketsDialog.vue';import TicketDescriptorProxy from 'pages/Ticket/Card/TicketDescriptorProxy.vue';import CustomerDescriptorProxy from 'pages/Customer/Card/CustomerDescriptorProxy.vue';import { useRoute } from 'vue-router';Al cargar esta seccion desaparece el buscardor de la parte superior, en basic-data sí aparece
Corregido
77e29a2b87.@ -0,0 +432,4 @@Warehouse: AlmacénPackages: BultosPackaging: EncajadoConfirm removal from route: Remover de la rutausar el verbo quitar en lugar de remover
Corregido
77e29a2b87.@ -0,0 +433,4 @@Packages: BultosPackaging: EncajadoConfirm removal from route: Remover de la rutaAre you sure you want to remove this ticket from the route?: ¿Seguro que quieres remover este ticket de la ruta?usar el verbo quitar en lugar de remover
Corregido
77e29a2b87.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