#5186 create parking section #197

Merged
jorgep merged 29 commits from 5186-CreateParkingSection into dev 2024-03-27 08:42:33 +00:00
4 changed files with 17 additions and 3 deletions
Showing only changes of commit b5edc98732 - Show all commits

View File

@ -706,11 +706,15 @@ export default {
code: 'Code',
pickingOrder: 'Picking order',
sector: 'Sector',
row: 'Row',
column: 'Column',
},
basicData: {
code: 'Code',
pickingOrder: 'Picking order',
sector: 'Sector',
row: 'Row',
column: 'Column',
},
},
invoiceIn: {

View File

@ -763,10 +763,14 @@ export default {
summary: {
code: 'Código',
pickingOrder: 'Orden de recogida',
row: 'Fila',
column: 'Columna',
},
basicData: {
code: 'Código',
pickingOrder: 'Orden de recogida',
row: 'Fila',
column: 'Columna',
},
},
invoiceIn: {

View File

@ -15,7 +15,7 @@ const parkingId = route.params?.id || null;
const sectors = ref([]);
const filter = {
fields: ['sectorFk', 'code', 'pickingOrder'],
fields: ['sectorFk', 'code', 'pickingOrder', 'row', 'column'],
include: [{ relation: 'sector', scope: { fields: ['id', 'description'] } }],
};
</script>
@ -28,7 +28,7 @@ const filter = {
auto-load
/>
<VnSubToolbar />
<FormModel :url="`Parkings/${parkingId}`" model="parking" :filter="filter">
<FormModel :url="`Parkings/${parkingId}`" model="parking" :filter="filter" auto-load>
<template #form="{ data }">
<VnRow>
<VnInput v-model="data.code" :label="t('parking.basicData.code')" />
@ -37,6 +37,10 @@ const filter = {
:label="t('parking.basicData.pickingOrder')"
/>
</VnRow>
<VnRow>
<VnInput v-model="data.row" :label="t('parking.basicData.row')" />
<VnInput v-model="data.column" :label="t('parking.basicData.column')" />
</VnRow>
<VnRow>
jorgep marked this conversation as resolved Outdated

El campo debería ser requerido si al enviar nulo da error.

O cambiar el mensaje de error en salix para que sea mas informativo al usuario y no de pie a pensar que algo está hecho mal.

El campo debería ser requerido si al enviar nulo da error. O cambiar el mensaje de error en salix para que sea mas informativo al usuario y no de pie a pensar que algo está hecho mal.

Esto habría que aplicarlo a nivel de FormModel, actualmente pasa en cualquier sección que inserte un campo que es obligatorio en bd, yo crearía tarea para hacer refactor de formModel. De momento, lo soluciono con la prop is-clearable="false". Pero que baraje @alexm y @jgallego si crear tarea.

Esto habría que aplicarlo a nivel de FormModel, actualmente pasa en cualquier sección que inserte un campo que es obligatorio en bd, yo crearía tarea para hacer refactor de formModel. De momento, lo soluciono con la prop is-clearable="false". Pero que baraje @alexm y @jgallego si crear tarea.
<VnSelectFilter
jorgep marked this conversation as resolved Outdated
Outdated
Review

Al final un VnRow con un unico elemento se veia bien?

Al final un VnRow con un unico elemento se veia bien?

sí, te aplica el flex 1, ocupando todo el ancho

sí, te aplica el flex 1, ocupando todo el ancho
v-model="data.sectorFk"

View File

@ -17,7 +17,7 @@ const { t } = useI18n();
const entityId = computed(() => $props.id || route.params.id);
Review

Yo valoraría crear un archivo general con los filtros de las cards de cada sección, porque si no hay que escribir el filtro. Si entramos en la sección summary no hay problema, se puede recoger el filtro desde la store, pero si abrimos el summaryProxy o popup, este no pasa por card y no tiene acceso al descriptor.

Yo valoraría crear un archivo general con los filtros de las cards de cada sección, porque si no hay que escribir el filtro. Si entramos en la sección summary no hay problema, se puede recoger el filtro desde la store, pero si abrimos el summaryProxy o popup, este no pasa por card y no tiene acceso al descriptor.
const filter = {
Outdated
Review

No pot gastar les dades que hi han en la store? les seteges en parkingCard

No pot gastar les dades que hi han en la store? les seteges en parkingCard

Sí pero hay que modificar la lógica de CardSummary, voy a dejarme esto en pendiente. Hoy subiré la rama con los cambios aplicados para que puedas ver como lo he hecho en el descriptor. No altera el funcionamiento del resto de la página.

Sé que te refieres a importar arrayData en parking summary y usarlo en CardSummary pero eso no tiene sentido porque sigues haciendo la llamada en CardSummary para usar los datos. Yo esto lo abordaría en una tarea. Avisame si quieres que lo miremos en llamada mañana.

Sí pero hay que modificar la lógica de CardSummary, voy a dejarme esto en pendiente. Hoy subiré la rama con los cambios aplicados para que puedas ver como lo he hecho en el descriptor. No altera el funcionamiento del resto de la página. Sé que te refieres a importar arrayData en parking summary y usarlo en CardSummary pero eso no tiene sentido porque sigues haciendo la llamada en CardSummary para usar los datos. Yo esto lo abordaría en una tarea. Avisame si quieres que lo miremos en llamada mañana.

Tras hablar con Alex , decidimos hacer un refactor en CardSummary para que use el arrayData e intentarlo con FormModel también.

Tras hablar con Alex , decidimos hacer un refactor en CardSummary para que use el arrayData e intentarlo con FormModel también.

El formModel lo paso a otra tarea https://redmine.verdnatura.es/issues/7135, que se me está alargando la tarea,

El formModel lo paso a otra tarea https://redmine.verdnatura.es/issues/7135, que se me está alargando la tarea,
fields: ['sectorFk', 'code', 'pickingOrder'],
fields: ['sectorFk', 'code', 'pickingOrder', 'row', 'column'],
include: [{ relation: 'sector', scope: { fields: ['id', 'description'] } }],
};
</script>
@ -43,6 +43,8 @@ const filter = {
:label="t('parking.summary.sector')"
:value="parking.sector?.description"
/>
<VnLv :label="t('parking.summary.row')" :value="parking.row" />
<VnLv :label="t('parking.summary.column')" :value="parking.column" />
</QCard>
</template>
</CardSummary>