feature/EntriesCorrections #177
jsegarra
alexm
Labels
No Milestone
No Assignees
4 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix-front#177
Loading…
Reference in New Issue
No description provided.
Delete Branch ":feature/EntriesCorrections"
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/6796
@ -0,0 +270,4 @@<template #body-cell-picture="{ row }"><QTd><QImg:src="`/api/Images/catalog/50x50/${row.itemFk}/download?access_token=${token}`"Revisar proporciones
Corregido.
Commit:
584511bc62@ -0,0 +249,4 @@<QSpace /><div id="st-actions"></div></QToolbar><!-- <QDrawer v-model="stateStore.rightDrawer" side="right" :width="256" show-if-above>Si no se usa o es un comentario, eliminar
Lo había dejado comentado porque es donde tendrían que ir los filtros laterales cuando terminemos de definir que hacer con ellos, pero igualmente eliminé el comentario:
Commit:
7089202fe1@ -0,0 +292,4 @@</QTd></template><template #body-cell-entryFk="{ row }"><QTd @click.stop="">Si no se maneja el evento, eliminar
El
@click.stopse utiliza para evitar la propagacion del evento a la row y que te direccione aEntry buys, por lo tanto es necesario, lo que si hice es eliminar el=""que no era necesarioCommit:
5280a4fa2d@ -43,3 +44,2 @@}),event: (props) => ({'keyup.enter': () => saveChange(props.row),event: (colField, props) => ({8 veces el mismo código para keyenter y blur.
Propuesta, sustituir las ocurrencias por uno genérico
Commit cambios:
2de02723baNueva mejora aplicada.
Commit:
5da5bb2047@ -0,0 +20,4 @@const { t } = useI18n();const { notify } = useNotify();const fieldsOptions = [@alexm Diría que en un componente genérico no definimos valores para un modulo en concreto, puedes confirmar.
Esto se pasaría por parámetro del componente no?
Exacto usar directamente t(...)
Se pasó la data clave a props para poder reutilizar en un futuro.
Commit:
ddcbc5d8d7@ -8,6 +8,8 @@ import FormModel from 'components/FormModel.vue';import VnRow from 'components/ui/VnRow.vue';import VnInput from 'src/components/common/VnInput.vue';import VnSelectFilter from 'src/components/common/VnSelectFilter.vue';import VnSelectDialog from 'src/components/common/VnSelectDialog.vue';Los checkbox del final, en español no tienen los mismo valores.
@alexm puedes confirmar si es correcto este cambio de nombre?
@ -83,3 +89,3 @@</div><div class="col"><VnSelectFilter<VnSelectDialogEn salix, para la entrada 3 y "Alm. salida", hay un registro que si pulsas en el id(está en azul), te muestra un popup.
En lilium, ese nuevo popup no se muestra y te cierra el dialogo
Corregido.
Commit:
a7e653cbce@ -49,6 +49,7 @@ onMounted(() => {:label="t('entry.notes.observationType')"Las opciones aparecen desordenadas. Es verdad que en salix vienen ordenadas de back, y en lilium no.
Propuesta, podemos hacer que estén ordenadas alfabéticamente?
Ordenamiento aplicado.
Commit:
28790705ea@ -29,3 +29,4 @@const originalRowDataCopy = ref(null);const tableColumnComponents = computed(() => ({item: {Intuyo que esta es la configuración de la columna Articulo de la tabla para entry/:id/buys
Falta itemDescriptor
Aplicado.
Commit:
0525f5fdd8@ -52,3 +52,4 @@:disable="!!row.id"option-label="description"option-value="id"hide-selectedPara el campo observation falta rules
Corregido.
Commit:
cde41ab337@ -284,6 +331,7 @@ const importBuys = () => {ref="entryBuysPaginateRef"data-key="EntryBuys":url="`Entries/${route.params.id}/getBuys`"@on-fetch="copyOriginalRowsData($event)"Podemos hacer desaprecer los labels de los inputs, como está hecho para embalaje.
Queda redundante tener el nombre de la columna en el input.
¿Que opinas @alexm ?
Sii es un mejor enfoque
Aplicado.
Commit:
86bbc7cfd6http://localhost:9000/#/entry/list el order deberia ser un array si no no funciona bien
en http://localhost:9000/#/entry/2/basic-data al usar el embudo de Travel y seleccionar una opción pone el numero y si guardas luego se rompe la app
En /buys al importar en item falta
En entry/latest-buys falta el filtro de la derecha
@ -57,2 +78,4 @@@on-data-saved="redirectToEntryBasicData"><template #form="{ data, validate }"><VnRow class="row q-gutter-md q-mb-md">Veo demasiado padding entre campos y arriba y abajo
Corregido.
Commit:
67e21303741> http://localhost:9000/#/entry/list el order deberia ser un array si no no funciona bien
2> en http://localhost:9000/#/entry/2/basic-data al usar el embudo de Travel y seleccionar una opción pone el numero y si guardas luego se rompe la app
3> En /buys al importar en item falta
4> En entry/latest-buys falta el filtro de la derecha
@alexm
1> Corregido
2> Esto ya funciona, lo único que si hay travels que cuando los seleccionas aparece el número en lugar del nombre del travel, esto pasa porque los resultados de la búsqueda ofrecen resultados que no contienen las opciones del
InputSelect, si observamos que pasa en salix en estos casos lo que sucede es lo mismo pero en lugar de mostrar el número, muestra el input vacío pero te deja guardar la información igualmente.3> Aplicado
4> Esto quedamos en crear una tarea y aplicar los filtros cuando se termine de definir que hacer con ellos para no bloquear todas las otras correcciones del modulo.
Issue: Crear filtros entry latest buys
Commit:
f6b0404cb4New commits pushed, approval review dismissed automatically according to repository settings