#7353 fine tunning monitors #624

Merged
jorgep merged 34 commits from 7353-fineTunningMonitor into dev 2024-09-02 07:33:43 +00:00
Member
No description provided.
jorgep added 1 commit 2024-08-16 11:54:42 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
a13ea900ce
fix: refs #7353 salesClientTable
jorgep added 2 commits 2024-08-16 13:18:35 +00:00
jorgep added 1 commit 2024-08-16 16:10:13 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
bd9aea767a
fix: refs #7353 salesorderTable
jorgep added 1 commit 2024-08-19 08:14:06 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
ada1c948f6
feat: refs #7353 split sections
jorgep added 1 commit 2024-08-19 08:30:05 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
9a4a51fadb
refactor: refs #7353 simplify code
jorgep added 2 commits 2024-08-19 12:57:37 +00:00
jorgep added 1 commit 2024-08-19 14:02:35 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
1ec7b7a8c1
fix: refs #7353 css
jorgep added 1 commit 2024-08-19 14:03:32 +00:00
jorgep reviewed 2024-08-19 14:05:46 +00:00
src/css/app.scss Outdated
@ -257,7 +257,6 @@ input::-webkit-inner-spin-button {
}
td {
font-size: 11pt;
border-top: 1px solid var(--vn-page-color);
Author
Member

rompía el estilo.

rompía el estilo.
jorgep reviewed 2024-08-19 14:07:17 +00:00
Author
Member

Es una fn no un evt personalizado.

Es una fn no un evt personalizado.
jorgep added the
CR / Tests passed
label 2024-08-19 14:16:25 +00:00
jorgep requested review from jsegarra 2024-08-19 14:17:08 +00:00
jorgep changed title from WIP: #7353 fine tunning monitors to #7353 fine tunning monitors 2024-08-19 14:17:14 +00:00
jorgep added 1 commit 2024-08-20 10:14:09 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
03baef8231
fix: refs #7353 css
jorgep added 1 commit 2024-08-21 09:36:02 +00:00
jorgep added 1 commit 2024-08-22 11:41:57 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
b0c27027c0
chore: refs #7353 use Vue component nomenclature
jorgep added 1 commit 2024-08-23 07:11:50 +00:00
Author
Member

@jsegarra No revisar aún.

@jsegarra No revisar aún.
jorgep added 1 commit 2024-08-23 10:43:33 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
fdacdf1a12
feat: refs #7353 add filter & fix customTags
jorgep added 1 commit 2024-08-23 13:56:00 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
1bbc6c1840
chore: refs #7353 fix warnings
jorgep added 1 commit 2024-08-27 07:55:57 +00:00
jorgep added 1 commit 2024-08-27 07:59:55 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
4491d83a7a
feat: refs #7353 add locale
jorgep added 1 commit 2024-08-28 07:41:15 +00:00
jorgep added 1 commit 2024-08-28 10:14:04 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
1312dbe2af
fix: refs #7353 iron out filter
jorgep reviewed 2024-08-28 10:21:54 +00:00
@ -25,3 +25,3 @@
default: true,
},
unRemovableParams: {
unremovableParams: {
Author
Member

Unremovable es una palabra.

Unremovable es una palabra.
jorgep reviewed 2024-08-28 10:22:07 +00:00
@ -180,10 +180,10 @@ const tagsList = computed(() => {
});
const tags = computed(() => {
return tagsList.value.filter((tag) => !($props.customTags || []).includes(tag.key));
Author
Member

esto no iba.

esto no iba.
jorgep reviewed 2024-08-28 10:29:22 +00:00
@ -10,6 +10,10 @@ const props = defineProps({
type: String,
required: true,
},
class: {
Author
Member

Si no da warning en consola

Si no da warning en consola
jorgep reviewed 2024-08-28 10:30:19 +00:00
@ -694,0 +673,4 @@
tr,
th,
td {
border-color: black;
Author
Member

así sigue habiendo borde.

así sigue habiendo borde.
jorgep reviewed 2024-08-28 10:30:24 +00:00
src/css/app.scss Outdated
@ -253,7 +253,6 @@ input::-webkit-inner-spin-button {
}
td {
font-size: 11pt;
border-top: 1px solid var(--vn-page-color);
Author
Member

Ya no hace falta,

Ya no hace falta,
jorgep reviewed 2024-08-28 10:30:47 +00:00
@ -2,3 +2,3 @@
const minHour = new Date(value);
minHour.setHours(0, 0, 0, 0);
const maxHour = new Date();
const maxHour = new Date(value);
Author
Member

Si no te pone hasta la fecha de hoy.

Si no te pone hasta la fecha de hoy.
jorgep added 1 commit 2024-08-28 10:46:38 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
495cd2cc2f
fix: refs #7353 iron out ticket table
jsegarra requested changes 2024-08-28 18:48:32 +00:00
Dismissed
@ -0,0 +129,4 @@
},
{
label: t('salesTicketsTable.practical'),
name: 'practicalHour',
Member

Revisamos porque en salix esta columna si tiene filtro

Revisamos porque en salix esta columna si tiene filtro
Author
Member

El filtro no iba en salix y en VnTable no funciona el filtro de hora.

El filtro no iba en salix y en VnTable no funciona el filtro de hora.
@ -0,0 +145,4 @@
},
{
label: t('salesTicketsTable.province'),
name: 'provinceFk',
Member

Podemos dartle un poco mas de espacio a provincia y comprimir fragil?

Podemos dartle un poco mas de espacio a provincia y comprimir fragil?
Author
Member

comprimir fragil no, porque cuando se ordena por esa columna, se corta. A provincia se le puede dar más espacio.

comprimir fragil no, porque cuando se ordena por esa columna, se corta. A provincia se le puede dar más espacio.
jorgep marked this conversation as resolved
@ -0,0 +344,4 @@
:right-search="false"
default-mode="table"
auto-load
:row-click="({ id }) => openTab(id)"
Member

hay alguna opción para esconde el modo card?

hay alguna opción para esconde el modo card?
Author
Member

El modo card se puede deshabilitar, pero, esconder el icono para cambiar a este no.

El modo card se puede deshabilitar, pero, esconder el icono para cambiar a este no.
jorgep marked this conversation as resolved
@ -0,0 +349,4 @@
:user-params="{ from, to, scopeDays: 1 }"
>
<template #top-left>
<QBtn
Member

Añadir tooltip.

Añadir tooltip.
jorgep marked this conversation as resolved
@ -0,0 +441,4 @@
<template #column-salesPersonFk="{ row }">
<QTd class="no-padding" @click.stop.prevent :title="row.userName">
<span class="link">{{ row.userName }}</span>
<WorkerDescriptorProxy :id="row.salesPersonFk" />
Member

Creo que falta añadir dashIfEmpty

Creo que falta añadir dashIfEmpty
jorgep marked this conversation as resolved
jsegarra requested changes 2024-08-28 19:00:23 +00:00
Dismissed
jsegarra left a comment
Member

Esta imagen corresponde a la nota #624 (comment)
image

Esta imagen corresponde a la nota https://gitea.verdnatura.es/verdnatura/salix-front/pulls/624#issuecomment-61148 ![image](/attachments/4d4d26fd-cdef-439f-9d0e-a34586e815aa)
@ -0,0 +56,4 @@
field: 'dated',
align: 'left',
columnFilter: false,
format: (row) => toDateFormat(row.dated),
Member

Revisamos el formateo ya que es dd/mm/yy

Revisamos el formateo ya que es dd/mm/yy
jorgep marked this conversation as resolved
@ -0,0 +125,4 @@
class="q-px-none"
>
<template #top-left>
<VnRow>
Member

PAra los 2 VnInputDate aplicamos el mismo style.
Usamos class

PAra los 2 VnInputDate aplicamos el mismo style. Usamos class
Author
Member

Te lo mejoro:
.full-width .vn-row > * { flex: 0.4; }

Te lo mejoro: `.full-width .vn-row > * { flex: 0.4; }`
jorgep marked this conversation as resolved
@ -0,0 +126,4 @@
>
<template #top-left>
<VnRow>
<VnInputDate v-model="from" label="From" dense style="flex: 0.4" />
Member

Traducir

Traducir
jorgep marked this conversation as resolved
@ -0,0 +127,4 @@
<template #top-left>
<VnRow>
<VnInputDate v-model="from" label="From" dense style="flex: 0.4" />
<VnInputDate v-model="to" label="To" dense style="flex: 0.4" />
Member

Traducir

Traducir
jorgep marked this conversation as resolved
@ -0,0 +137,4 @@
</QTd>
</template>
<template #column-clientFk="{ row }">
<QTd>
Member

El texto no aparece completo
"Bruce Way..." es lo que se ve

El texto no aparece completo "Bruce Way..." es lo que se ve
Author
Member

Le he quitado el padding, con el espacio que hay poco más se puede hacer. la tabla tiene el ancho que toca. Ahora los nombre de esa longitud y un poco más se ven.

Le he quitado el padding, con el espacio que hay poco más se puede hacer. la tabla tiene el ancho que toca. Ahora los nombre de esa longitud y un poco más se ven.
@ -0,0 +118,4 @@
window.open(`#/order/${id}/summary`, '_blank', 'noopener, noreferrer');
</script>
<template>
<VnTable
Member

Pregunta, En Salix es order: date_makeDESC y aqui es date_send DESC, ha cambiado el criterio?

Pregunta, En Salix es order: date_makeDESC y aqui es date_send DESC, ha cambiado el criterio?
Author
Member

Se lo consulté a J. Gallego y acordamos dejarlo como está ahora.

Se lo consulté a J. Gallego y acordamos dejarlo como está ahora.
jorgep marked this conversation as resolved
@ -0,0 +146,4 @@
color="primary"
dense
flat
@click="$refs.table.reload()"
Member

tooltip

tooltip
jorgep marked this conversation as resolved
@ -0,0 +181,4 @@
<template #column-clientFk="{ row }">
<QTd>
<span class="link" v-text="row.clientName" :title="row.clientName" />
Member

Comportamiento exacto que en column-comercialFk

Comportamiento exacto que en column-comercialFk
jorgep marked this conversation as resolved
@ -0,0 +187,4 @@
</template>
<template #column-salesPersonFk="{ row }">
<span class="link" v-text="row.salesPerson" />
Member

mmm...revisa el comportamiento porque al hacer click te abre una nueva pestña en vez de mostrarte el descriptor. si vuelves a monitores si que se ve el descriptor

mmm...revisa el comportamiento porque al hacer click te abre una nueva pestña en vez de mostrarte el descriptor. si vuelves a monitores si que se ve el descriptor
jorgep marked this conversation as resolved
jsegarra requested changes 2024-08-28 19:02:27 +00:00
Dismissed
jsegarra left a comment
Member

Esta imagen corresponde a este comentario #624 (comment)
image

Esta imagen corresponde a este comentario https://gitea.verdnatura.es/verdnatura/salix-front/pulls/624#issuecomment-61156 ![image](/attachments/e17c88b2-cbd4-4e95-8857-0f41125b44d7)
@ -0,0 +67,4 @@
name: 'salesPersonFk',
align: 'left',
optionFilter: 'firstName',
columnFilter: {
Member

Cuando uso este filtro o el de cliente, me ocurre que al pulsar sobre la X, VnTable va bien, pero me aparece un toast de error sin texto
Te ocurre? Es poblema de VnTable?

Cuando uso este filtro o el de cliente, me ocurre que al pulsar sobre la X, VnTable va bien, pero me aparece un toast de error sin texto Te ocurre? Es poblema de VnTable?
Author
Member

Sí, a veces. Pero esto es cosa de VnTable. Lo he probado en ClaimList y pasa igual.

Sí, a veces. Pero esto es cosa de VnTable. Lo he probado en ClaimList y pasa igual.
jorgep added 1 commit 2024-08-29 07:29:45 +00:00
jorgep added 1 commit 2024-08-29 09:40:46 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
966667bab7
refactor: refs #7353 tickets correction
jorgep added 3 commits 2024-08-29 10:32:04 +00:00
jorgep added 1 commit 2024-08-29 11:04:59 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
58e47b3f24
refactor: refs #7353 order correction
jorgep requested review from jsegarra 2024-08-29 11:05:12 +00:00
jorgep added 1 commit 2024-08-29 11:06:40 +00:00
jorgep added 1 commit 2024-08-29 14:40:39 +00:00
jorgep added 1 commit 2024-08-29 14:51:59 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
6062c2d845
feat: refs #7353 salesPerson nickname & id
jorgep added 1 commit 2024-08-29 14:54:14 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
90b2cf09b9
feat: refs #7353 add right icons
jorgep added 1 commit 2024-08-30 09:40:59 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
f9a509ac67
feat: refs #7353 add no one opt
jsegarra requested changes 2024-08-30 21:56:23 +00:00
Dismissed
@ -0,0 +19,4 @@
const formatFrom = setHours(from.value, 'from');
const formatTo = setHours(to.value, 'to');
if (!formatFrom && formatTo)
Member

Veo mucho Object.assign.
Crees que podemos dejar esto debajo del if...else..if y que el valor de v.stamp sea una variable?

Veo mucho Object.assign. Crees que podemos dejar esto debajo del if...else..if y que el valor de v.stamp sea una variable?
jorgep marked this conversation as resolved
@ -0,0 +131,4 @@
</VnRow>
</template>
<template #column-salesPersonFk="{ row }">
<QTd class="no-padding">
Member

Revisa porque creo que le sobre el Qtd y le falta el @click.stop
O pones Qtd por el .no-padding?

Revisa porque creo que le sobre el Qtd y le falta el @click.stop O pones Qtd por el .no-padding?
Author
Member

Esta tabla no redirige a ningún sitio, no le hace falta el click.stop. El qTd lo quito.

Esta tabla no redirige a ningún sitio, no le hace falta el click.stop. El qTd lo quito.
jorgep marked this conversation as resolved
@ -0,0 +189,4 @@
</template>
<template #column-salesPersonFk="{ row }">
<QTd @click.stop>
Member

Está diferente a MonitorClients, por algún detalle en concreto que se me pase?

Está diferente a MonitorClients, por algún detalle en concreto que se me pase?
Author
Member

Las filas redirigen al hacer click. En clients no.

Las filas redirigen al hacer click. En clients no.
jorgep marked this conversation as resolved
@ -0,0 +28,4 @@
};
</script>
<template>
<FetchData url="Warehouses" auto-load @on-fetch="(data) => (warehouses = data)" />
Member

VnSelect podria tener esta url no?
No nos quitaremos la dependencia de FetchData pero ya que tenemos esa funcionalidad...
Y realmente warehouses solo se repite 4 veces y nos quitamos la variable que no tiene recorrido en este componente

VnSelect podria tener esta url no? No nos quitaremos la dependencia de FetchData pero ya que tenemos esa funcionalidad... Y realmente warehouses solo se repite 4 veces y nos quitamos la variable que no tiene recorrido en este componente
Author
Member

Solo hay 88 registros en la tabla warehouse, cuando son tablas pequeñas se acordó cargarla toda de golpe.

Solo hay 88 registros en la tabla warehouse, cuando son tablas pequeñas se acordó cargarla toda de golpe.
jorgep added 1 commit 2024-09-02 07:09:32 +00:00
jsegarra approved these changes 2024-09-02 07:29:48 +00:00
Dismissed
jorgep added 1 commit 2024-09-02 07:30:20 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
ed2167e199
refactor: refs #7353 ease logic
jorgep dismissed jsegarra’s review 2024-09-02 07:30:20 +00:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

jorgep requested review from jsegarra 2024-09-02 07:31:07 +00:00
jsegarra approved these changes 2024-09-02 07:31:28 +00:00
jorgep merged commit f0055d7b36 into dev 2024-09-02 07:33:43 +00:00
jorgep deleted branch 7353-fineTunningMonitor 2024-09-02 07:33:43 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: verdnatura/salix-front#624
No description provided.