HOTFIX: #6943 CustomerList form salesPersons options #790

Open
jsegarra wants to merge 76 commits from hotfix_newCustomer_SalesPerson into master
Member

He bajado 2 acciones del menú a botón a petición de Jesus. Estas acciones no muestran el modal dentro de cliente sino que abre una pestaña nueva en el listado de Tickets filtrados del mismo y el campo cliente preseleccionado. De igual manera para el boton de crear nuevo pedido/cesta

He bajado 2 acciones del menú a botón a petición de Jesus. Estas acciones no muestran el modal dentro de cliente sino que abre una pestaña nueva en el listado de Tickets filtrados del mismo y el campo cliente preseleccionado. De igual manera para el boton de crear nuevo pedido/cesta
jsegarra added 1 commit 2024-10-02 08:37:14 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
6c97f5eeb3
fix: CustomerList form salesPersons options
jsegarra added 1 commit 2024-10-02 08:38:56 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
aa41f0d826
fix: CustomerList form salesPersons options
jsegarra requested review from alexm 2024-10-02 08:39:02 +00:00
alexm reviewed 2024-10-02 08:43:51 +00:00
jsegarra added 1 commit 2024-10-03 22:27:59 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
02e7177dee
Merge branch 'master' into hotfix_newCustomer_SalesPerson
jsegarra added 3 commits 2024-10-03 23:23:33 +00:00
jsegarra added 1 commit 2024-10-03 23:27:24 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
0b752cdb0d
feat: #6942 Open TicketSale in new tab
jsegarra requested review from jgallego 2024-10-03 23:27:55 +00:00
jsegarra added 1 commit 2024-10-03 23:32:17 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
5770c344c8
feat: #6942 Create Ticket or Order with data filtered by clientId
jsegarra added 1 commit 2024-10-04 07:12:58 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
01764f4954
style: #refs CardSummary max-height
jgallego requested changes 2024-10-04 07:15:49 +00:00
Dismissed
@ -56,3 +56,3 @@
<template>
<QItem v-ripple clickable @click="openTicketCreateForm()">
<!-- <QItem v-ripple clickable @click="openTicketCreateForm()">
Owner

los comentarios no suben

los comentarios no suben
jsegarra marked this conversation as resolved
@ -98,2 +98,3 @@
icon: 'vn:lines',
action: ({ id }) => router.push({ params: { id }, name: 'TicketSale' }),
action: ({ id }) =>
window.open(
Owner

El otro día comenté contigo con Alex que el Windows open mejor no usarlo. aplica aqui?

El otro día comenté contigo con Alex que el Windows open mejor no usarlo. aplica aqui?
Author
Member

El viernes pasado comentamos que habia un composable openReport que hacia esto.
Para este caso concreto lo ideal seria usar la etiqueta a de HTML y la propiedad hRef, pero en este caso no aplica

El viernes pasado comentamos que habia un composable openReport que hacia esto. Para este caso concreto lo ideal seria usar la etiqueta a de HTML y la propiedad hRef, pero en este caso no aplica
jgallego marked this conversation as resolved
jsegarra added 1 commit 2024-10-04 07:17:52 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
d34a7b5839
style: #refs remove commentst
jsegarra requested review from jgallego 2024-10-04 07:23:27 +00:00
jgallego approved these changes 2024-10-04 08:01:06 +00:00
Dismissed
jsegarra added 1 commit 2024-10-04 17:51:10 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
9ddd7f9524
Merge branch 'master' into hotfix_newCustomer_SalesPerson
jsegarra added 4 commits 2024-10-04 20:29:20 +00:00
jsegarra dismissed jgallego’s review 2024-10-04 20:29:20 +00:00
Reason:

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

jsegarra added 1 commit 2024-10-07 18:20:01 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
a509a40d3d
Merge branch 'master' into hotfix_newCustomer_SalesPerson
jsegarra added 2 commits 2024-10-07 18:37:37 +00:00
alexm requested changes 2024-10-08 10:12:51 +00:00
Dismissed
@ -118,6 +118,7 @@ function existSummary(routes) {
.cardSummary {
width: 100%;
max-height: 70vh;
Member

Se quiere asi?

Se quiere asi?
Author
Member

Lo solicita Jesus Brocal ya que se queda un espacio muy pequeño para cerrar el modal. Es incomodo.
El cirterio para darle este valor es que el modal como limite, debe respetar el alto de la barra superior

Lo solicita Jesus Brocal ya que se queda un espacio muy pequeño para cerrar el modal. Es incomodo. El cirterio para darle este valor es que el modal como limite, debe respetar el alto de la barra superior
Member

Pero cardSummary se usa en los summary no deberia tener el alto limitado

Pero cardSummary se usa en los summary no deberia tener el alto limitado
jsegarra marked this conversation as resolved
@ -21,6 +22,7 @@ const tableRef = ref();
const agencyList = ref([]);
const addressesList = ref([]);
const clientId = ref();
Member

Si ya no se usa, quitar

Si ya no se usa, quitar
jsegarra marked this conversation as resolved
@ -137,3 +139,3 @@
title: t('ticketList.summary'),
icon: 'preview',
action: (row) => viewSummary(row.id, TicketSummary),
action: (row, evt) => {
Member

Sin hacer este codigo ya funciona asi no? (lo he probado en master)

Sin hacer este codigo ya funciona asi no? (lo he probado en master)
Author
Member

Antes tambien funcionaba, pero lo que se quiere es que el summary se pueda abrir en una pestaña nueva, con control, auqnue creo que seria mejor hacerlo con el click de la rueda. Lo reviso

Antes tambien funcionaba, pero lo que se quiere es que el summary se pueda abrir en una pestaña nueva, con control, auqnue creo que seria mejor hacerlo con el click de la rueda. Lo reviso
jsegarra marked this conversation as resolved
@ -0,0 +27,4 @@
Location: { val: '46000, Valencia(Province one), España', type: 'select' },
};
cy.fillInForm(data);
// cy.get(dialogInputs).eq(0).type('NAME 1');
Member

Quitar

Quitar
jsegarra marked this conversation as resolved
jsegarra added 4 commits 2024-10-08 22:34:08 +00:00
jsegarra added 1 commit 2024-10-08 22:41:15 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
e550f54314
fix: change type vnput
jsegarra added the
CR / Tests passed
label 2024-10-08 22:45:22 +00:00
jsegarra requested review from alexm 2024-10-08 22:45:35 +00:00
jsegarra added 2 commits 2024-10-10 10:17:28 +00:00
jsegarra added 1 commit 2024-10-10 14:03:37 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
1cafd79d67
fix: customerSummary agencyMode
jsegarra added 2 commits 2024-10-11 13:28:25 +00:00
alexm requested changes 2024-10-14 08:02:23 +00:00
Dismissed
@ -140,0 +144,4 @@
params: { id: row.id },
name: 'TicketCard',
}).href;
window.open(url, '_blank');
Member

Lo que queria decir es que esta funcionalidad ya esta hecha, funciona asi sin añadir ningun codigo

Lo que queria decir es que esta funcionalidad ya esta hecha, funciona asi sin añadir ningun codigo
Author
Member

Si vas a https://lilium.verdnatura.es/#/customer/34655/summary y le das a

  1. Ctrl+click al icono de lineas, no te abre nueva pestaña, te reemplaza donde estas. Lo que quieren es mantener la vista del cliente y en otra las lineas de ese ticket
  2. Ctrl+click al icono de summary, te muestra el summaryen esa ventana. Lo que quieren es que se te abra una nueva pestaña para mantener el listado de tickets

Yo lo he probado y no va en lilium.verdnatura.es Si quieres, envíame un video por privado

Si vas a https://lilium.verdnatura.es/#/customer/34655/summary y le das a 1. Ctrl+click al icono de lineas, no te abre nueva pestaña, te reemplaza donde estas. Lo que quieren es mantener la vista del cliente y en otra las lineas de ese ticket 2. Ctrl+click al icono de summary, te muestra el summaryen esa ventana. Lo que quieren es que se te abra una nueva pestaña para mantener el listado de tickets Yo lo he probado y no va en lilium.verdnatura.es Si quieres, envíame un video por privado
Member

No me deja subir video, pero en master funciona ya el ctrl + click para abrir en una nueva pestaña

No me deja subir video, pero en master funciona ya el ctrl + click para abrir en una nueva pestaña
jsegarra added 4 commits 2024-10-14 11:48:50 +00:00
jsegarra added 1 commit 2024-10-14 12:54:45 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
7bb8330302
feat: refs #6943 add order by id
jsegarra requested review from alexm 2024-10-14 14:11:30 +00:00
jsegarra reviewed 2024-10-14 14:12:26 +00:00
@ -28,3 +28,3 @@
'postalCode',
],
order: ['isDefaultAddress DESC', 'isActive DESC', 'nickname ASC'],
order: ['isDefaultAddress DESC', 'isActive DESC', 'id DESC', 'nickname ASC'],
Author
Member

Mejoramos la lista de resultados, haciendo que este ordenado por id respetando la opcion por defecto

Mejoramos la lista de resultados, haciendo que este ordenado por id respetando la opcion por defecto
jsegarra reviewed 2024-10-14 14:13:07 +00:00
@ -249,3 +249,3 @@
function updateStateParams() {
const newUrl = { path: route.path, query: { ...(route.query ?? {}) } };
newUrl.query[store.searchUrl] = JSON.stringify(store.currentFilter);
if (!route.path.endsWith('/summary'))
Author
Member

Así evitamos ahora y a futuro que si usamos VnTable en *summary, tengamos los filtros de la tabla

Así evitamos ahora y a futuro que si usamos VnTable en *summary, tengamos los filtros de la tabla
Member

Añadirle un searchUrl diferente a la tabla que se use, como ya hacemos en las otras secciones

Añadirle un searchUrl diferente a la tabla que se use, como ya hacemos en las otras secciones
Author
Member

Mi no entender.
No se elimina la linea

Mi no entender. No se elimina la linea
Member

Ya pero que la solucion seria mas en la tabla que haya en ese summary ponerle searchUrl="sumarry"
Mas que poner aqui una condicion especifica no?

Ya pero que la solucion seria mas en la tabla que haya en ese summary ponerle searchUrl="sumarry" Mas que poner aqui una condicion especifica no?
Author
Member

Los usuarios nos dicen que para compartir el summary de un cliente ahora les sale esto en el portapapeles http://localhost:9000/#/customer/1109/summary?tickets={%22filter%22:%22{%5C%22limit%5C%22:30,%5C%22include%5C%22:[{%5C%22relation%5C%22:%5C%22ticketState%5C%22,%5C%22scope%5C%22:{%5C%22fields%5C%22:[%5C%22stateFk%5C%22,%5C%22code%5C%22,%5C%22alertLevel%5C%22],%5C%22include%5C%22:{%5C%22relation%5C%22:%5C%22state%5C%22}}},{%5C%22relation%5C%22:%5C%22invoiceOut%5C%22,%5C%22scope%5C%22:{%5C%22fields%5C%22:[%5C%22id%5C%22]}},{%5C%22relation%5C%22:%5C%22agencyMode%5C%22,%5C%22scope%5C%22:{%5C%22fields%5C%22:[%5C%22name%5C%22]}}],%5C%22where%5C%22:{%5C%22clientFk%5C%22:%5C%221109%5C%22},%5C%22order%5C%22:%5C%22shipped+DESC,+id%5C%22,%5C%22skip%5C%22:0}%22}

Lo que estan haciendo a mano es seleccionar este texto http://localhost:9000/#/customer/1109/summary

Por tanto, si me dices que hay alguna manera de montar el summary si que se añada la parte de la query, por favor, indicame como. Porque hasta ahora, he probado quitando o poniendo un valor a search-url y siempre me pone table o el valor, respectivamente.
Caso de existo, entras en un cliente y en la URL tienes esto http://localhost:9000/#/customer/1109/summary

Los usuarios nos dicen que para compartir el summary de un cliente ahora les sale esto en el portapapeles http://localhost:9000/#/customer/1109/summary?tickets={%22filter%22:%22{%5C%22limit%5C%22:30,%5C%22include%5C%22:[{%5C%22relation%5C%22:%5C%22ticketState%5C%22,%5C%22scope%5C%22:{%5C%22fields%5C%22:[%5C%22stateFk%5C%22,%5C%22code%5C%22,%5C%22alertLevel%5C%22],%5C%22include%5C%22:{%5C%22relation%5C%22:%5C%22state%5C%22}}},{%5C%22relation%5C%22:%5C%22invoiceOut%5C%22,%5C%22scope%5C%22:{%5C%22fields%5C%22:[%5C%22id%5C%22]}},{%5C%22relation%5C%22:%5C%22agencyMode%5C%22,%5C%22scope%5C%22:{%5C%22fields%5C%22:[%5C%22name%5C%22]}}],%5C%22where%5C%22:{%5C%22clientFk%5C%22:%5C%221109%5C%22},%5C%22order%5C%22:%5C%22shipped+DESC,+id%5C%22,%5C%22skip%5C%22:0}%22} Lo que estan haciendo a mano es seleccionar este texto http://localhost:9000/#/customer/1109/summary Por tanto, si me dices que hay alguna manera de montar el summary si que se añada la parte de la query, por favor, indicame como. Porque hasta ahora, he probado quitando o poniendo un valor a search-url y siempre me pone table o el valor, respectivamente. Caso de existo, entras en un cliente y en la URL tienes esto http://localhost:9000/#/customer/1109/summary
jsegarra reviewed 2024-10-14 14:14:03 +00:00
@ -5,3 +10,3 @@
<template>
<VnInput v-bind="$attrs" v-model.number="model" type="number" />
<VnInput v-bind="$attrs" v-model.number="model" type="number" :step="step" />
Author
Member

Daba error en los inputs de tipo numero al introducir 2 decimales

Daba error en los inputs de tipo numero al introducir 2 decimales
jsegarra reviewed 2024-10-14 14:14:31 +00:00
@ -34,2 +36,2 @@
const requiredFieldRule = (val) => validations().required($attrs.required, val);
const requiredFieldRule = (val) => validations().required(isRequired.value, val);
const isRequired = computed(() => Object.keys($attrs).includes('required'));
Author
Member

Evitamos tener que hacer :required="true", ahora solo hace falta required

Evitamos tener que hacer :required="true", ahora solo hace falta required
jsegarra changed title from HOTFIX: CustomerList form salesPersons options to WIP: HOTFIX: CustomerList form salesPersons options 2024-10-14 14:15:25 +00:00
jsegarra added 1 commit 2024-10-14 21:25:34 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
4296a74be5
fix: refs #6943 CustomerBalance using VnFilter and show all options
jsegarra changed title from WIP: HOTFIX: CustomerList form salesPersons options to HOTFIX: CustomerList form salesPersons options 2024-10-14 21:28:40 +00:00
jsegarra added 1 commit 2024-10-14 21:28:49 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
ade288efc3
Merge branch 'master' into hotfix_newCustomer_SalesPerson
jsegarra added 1 commit 2024-10-16 13:43:38 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
f3d0dd37d2
Merge branch 'master' into hotfix_newCustomer_SalesPerson
jsegarra added 2 commits 2024-10-22 21:15:20 +00:00
alexm requested changes 2024-10-24 10:14:39 +00:00
Dismissed
@ -538,3 +550,3 @@
(btn.show && btn.show(row)) ?? true ? 'visible' : 'hidden'
}`"
@click="btn.action(row)"
@click="handleClick($event, btn, row)"
Member

Yo haria que te pase los datos a la función y en la función ya tenga la logica.

Yo haria que te pase los datos a la función y en la función ya tenga la logica.
Author
Member

Estas proponiendo que la lógica de la función handleClick esté en la definición de las acciones en CustomerSummaryTable?

Estas proponiendo que la lógica de la función handleClick esté en la definición de las acciones en CustomerSummaryTable?
Author
Member

De todas maneras he deshecho el cambio porque al parece faltó añadir row-click

De todas maneras he deshecho el cambio porque al parece faltó añadir row-click
@ -220,3 +220,3 @@
@click="search()"
></QBtn>
<QForm @submit="search" id="filterPanelForm">
<QForm @submit="search" id="filterPanelForm" @keyup.enter="search()">
Member

Pero esto ya lo tiene QForm no?

Pero esto ya lo tiene QForm no?
Author
Member

?Pero no está funcionando en master.
Al parecer en test si que va.
Los usuarios nos lo piden mucho, podemos eliminar la linea y...sabemos que se ha cambiado para que funcione correctamente?

?Pero no está funcionando en master. Al parecer en test si que va. Los usuarios nos lo piden mucho, podemos eliminar la linea y...sabemos que se ha cambiado para que funcione correctamente?
Member

Yo lo quitaria, puede que de problemas.
Prueba un filtro lateral que genere VnTable

Yo lo quitaria, puede que de problemas. Prueba un filtro lateral que genere VnTable
Author
Member

Se mantiene, todavia no está el evento que lo dispara

Se mantiene, todavia no está el evento que lo dispara
alexm requested changes 2024-10-24 10:16:53 +00:00
Dismissed
@ -70,3 +70,3 @@
default: false,
},
appendParams: {
Member

Usar la propiedad searchUrl a false para esto

Usar la propiedad searchUrl a false para esto
Author
Member

searchUrl es una propiedad de tipo String que tiene por defecto table.
Aunque no la declares, te pone table
Si la declaras no puedes decirle que valga false, null o undefined

searchUrl es una propiedad de tipo String que tiene por defecto table. Aunque no la declares, te pone table Si la declaras no puedes decirle que valga false, null o undefined
Member

Ni poniendo q el type sea [string, boolean]??

Ni poniendo q el type sea [string, boolean]??
Author
Member

Si esta solución la había planteado y probado, pero no me gustaba porque mezcla 2 tipos. Aunque veo que hay 2 resultados mas para esta combinación

Si esta solución la había planteado y probado, pero no me gustaba porque mezcla 2 tipos. Aunque veo que hay 2 resultados mas para esta combinación
Member

Uf es q lo veo mas facil de usar que añadir un parametro mas.
Creo que poniendo el if y pasando comillas vacias haria la funcion de booleano jajja

Uf es q lo veo mas facil de usar que añadir un parametro mas. Creo que poniendo el if y pasando comillas vacias haria la funcion de booleano jajja
@ -249,3 +249,3 @@
function updateStateParams() {
const newUrl = { path: route.path, query: { ...(route.query ?? {}) } };
newUrl.query[store.searchUrl] = JSON.stringify(store.currentFilter);
if (store.appendParams)
Member

Y aqui sea if(store.searchUrl)

Y aqui sea if(store.searchUrl)
Author
Member

lo coge como String no como boolean

lo coge como String no como boolean
jsegarra added 2 commits 2024-10-24 22:11:19 +00:00
gitea/salix-front/pipeline/pr-master There was a failure building this commit Details
e9c3b32b8a
revert: #6943 change related to keyboard
jsegarra requested review from alexm 2024-10-24 22:11:51 +00:00
jsegarra added 3 commits 2024-10-25 07:50:45 +00:00
alexm requested changes 2024-10-28 06:27:45 +00:00
Dismissed
@ -64,1 +64,3 @@
return !isSummary.value && route.meta.moduleName && exist;
const isDialog = document.querySelectorAll('[role="dialog"]').length > 0;
// const isSameModule = `${route.meta.moduleName}Summary` === props.dataKey;
Member

Quitar comment y console.log

Quitar comment y console.log
jsegarra marked this conversation as resolved
@ -143,3 +171,1 @@
amount: 0,
code: companies.value.find((c) => c.id === companyId.value)?.code,
};
async function getClientRisk() {
Member

Revisar bien la funcionalidad de este archivo, se usa en otro sitio

Revisar bien la funcionalidad de este archivo, se usa en otro sitio
Author
Member

he creado un composable

he creado un composable
@ -55,4 +43,3 @@
</script>
<template>
<QItem v-ripple clickable @click="openTicketCreateForm()">
Member

Se quitan?

Se quitan?
Author
Member

Se mueve la funcionalidad del DescriptorMenu a botón en Descriptor actions

Se mueve la funcionalidad del DescriptorMenu a botón en Descriptor actions
@ -25,6 +25,11 @@ const $props = defineProps({
const entityId = computed(() => $props.id || route.params.id);
const customer = computed(() => summary.value.entity);
const summary = ref();
const clientUrl = ref();
Member

No se usa no?

No se usa no?
Author
Member

En quasar build no sale el warning.
Se usa en el onMounted

En quasar build no sale el warning. Se usa en el onMounted
jsegarra marked this conversation as resolved
jsegarra added 2 commits 2024-10-28 09:32:55 +00:00
jsegarra added 1 commit 2024-10-28 09:32:58 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
f8729c29eb
Merge branch 'master' into hotfix_newCustomer_SalesPerson
jsegarra requested review from alexm 2024-10-28 09:33:02 +00:00
alexm approved these changes 2024-10-28 09:43:23 +00:00
Dismissed
jsegarra added 2 commits 2024-10-29 00:07:41 +00:00
jsegarra dismissed alexm’s review 2024-10-29 00:07:41 +00:00
Reason:

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

jsegarra added 1 commit 2024-10-29 00:21:55 +00:00
gitea/salix-front/pipeline/pr-master There was a failure building this commit Details
dec2db2baa
feat: #6943 minor changes
jsegarra added 1 commit 2024-10-29 01:30:57 +00:00
gitea/salix-front/pipeline/pr-master There was a failure building this commit Details
39963e4ec9
perf: remove required logic for some inputs
jsegarra added 1 commit 2024-10-29 01:38:48 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
987c28e745
test: fix arrayData
jsegarra added 1 commit 2024-11-08 08:23:31 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
1c3811723e
feat: remove appendParams feature
jsegarra requested review from alexm 2024-11-08 08:23:48 +00:00
alexm approved these changes 2024-11-12 10:32:10 +00:00
Dismissed
alexm left a comment
Member

Arreglar conflictos

Arreglar conflictos
jsegarra added 2 commits 2024-11-12 12:29:57 +00:00
jsegarra dismissed alexm’s review 2024-11-12 12:29:57 +00:00
Reason:

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

jsegarra added 1 commit 2024-11-12 12:30:50 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
b6a3dfa45a
feat: remove cli warnings
jsegarra added 1 commit 2024-11-12 12:56:17 +00:00
gitea/salix-front/pipeline/pr-master There was a failure building this commit Details
20f794b553
style: reduce padding actions cardDescriptor
jsegarra requested review from alexm 2024-11-12 13:11:51 +00:00
Member

Sigue habiendo uno jeje

Sigue habiendo uno jeje
jsegarra changed title from HOTFIX: CustomerList form salesPersons options to WIP: HOTFIX: CustomerList form salesPersons options 2024-11-12 13:13:06 +00:00
Author
Member

Los test e2e no pasan

Los test e2e no pasan
jsegarra added 1 commit 2024-11-12 13:48:56 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
236df82c90
Merge branch 'master' into hotfix_newCustomer_SalesPerson
jsegarra changed title from WIP: HOTFIX: CustomerList form salesPersons options to HOTFIX: CustomerList form salesPersons options 2024-11-12 13:50:30 +00:00
jsegarra added 3 commits 2024-11-13 11:05:14 +00:00
jsegarra changed title from HOTFIX: CustomerList form salesPersons options to HOTFIX: #6943 CustomerList form salesPersons options 2024-11-13 11:22:18 +00:00
jsegarra added 1 commit 2024-11-13 11:23:47 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
367343d2a8
feat: #6943 apply color based on condition
alexm approved these changes 2024-11-13 11:26:16 +00:00
Dismissed
jsegarra added 1 commit 2024-11-13 11:30:05 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
e837988744
Merge branch 'master' into hotfix_newCustomer_SalesPerson
jsegarra added 2 commits 2024-11-13 11:39:42 +00:00
jsegarra added 2 commits 2024-11-13 11:52:16 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
6ac52e0b5f
feat: remove button
jsegarra dismissed alexm’s review 2024-11-13 11:52:16 +00:00
Reason:

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

jsegarra added 1 commit 2024-11-13 11:53:42 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
6c9406cfb0
perf: #6943 TicketList
jsegarra added 1 commit 2024-11-13 12:39:33 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
775b14ed65
fix: #6943 CustomersummaryTable route click
jsegarra added 1 commit 2024-11-14 08:05:43 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
683edfbf56
Merge branch 'master' into hotfix_newCustomer_SalesPerson
jsegarra added 2 commits 2024-11-14 10:17:51 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
de2a926058
feat: move buttons to DescriptorMenu
jsegarra added 1 commit 2024-11-14 21:53:34 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
08a4838007
Merge branch 'master' into hotfix_newCustomer_SalesPerson
jsegarra requested review from alexm 2024-11-15 08:04:34 +00:00
alexm requested changes 2024-11-15 08:27:53 +00:00
Dismissed
@ -168,6 +172,27 @@ const setData = (entity) => (data.value = useCardDescription(entity?.name, entit
>
<QTooltip>{{ t('Customer ticket list') }}</QTooltip>
</QBtn>
<!-- <QBtn
Member

Comentado?

Comentado?
jsegarra marked this conversation as resolved
@ -179,6 +204,26 @@ const setData = (entity) => (data.value = useCardDescription(entity?.name, entit
>
<QTooltip>{{ t('Customer invoice out list') }}</QTooltip>
</QBtn>
<!-- <QBtn
Member

Comentado?

Comentado?
jsegarra marked this conversation as resolved
@ -28,0 +28,4 @@
const clientUrl = ref();
onMounted(async () => {
clientUrl.value = (await getUrl('client/')) + entityId.value + '/';
Member

no se usa?

no se usa?
jsegarra marked this conversation as resolved
@ -180,3 +187,3 @@
search-url="tickets"
:without-header="true"
auto-load
:row-click="rowClick"
Member

Creo que poniendo redirect ya lo hace no?

Creo que poniendo redirect ya lo hace no?
Author
Member

espectacular

espectacular
@ -253,3 +254,4 @@
cy.get(selector).should('have.text', expectedValue);
});
Cypress.Commands.add('openActionDescriptor', (opt) => {
Member

No se usa?

No se usa?
Author
Member

Este no

Este no
jsegarra marked this conversation as resolved
@ -256,1 +261,4 @@
1;
});
Cypress.Commands.add('openActionsDescriptor', () => {
Member

No se usa?

No se usa?
Author
Member

Se usa en 3 archivos, ticketDescriptor e invoiceInDescriptor

Se usa en 3 archivos, ticketDescriptor e invoiceInDescriptor
jsegarra marked this conversation as resolved
@ -257,3 +265,4 @@
cy.get('.header > :nth-child(3) > .q-btn__content > .q-icon').click();
});
Cypress.Commands.add('clickButtonsDescriptor', (id) => {
Member

Revisar que se usen los metodos

Revisar que se usen los metodos
Author
Member

Se usa en clientList

Se usa en clientList
jsegarra marked this conversation as resolved
jsegarra added 4 commits 2024-11-15 09:51:23 +00:00
jsegarra requested review from alexm 2024-11-15 09:51:32 +00:00
alexm approved these changes 2024-11-15 10:11:47 +00:00
All checks were successful
gitea/salix-front/pipeline/pr-master This commit looks good
Required
Details
This pull request has changes conflicting with the target branch.
  • src/pages/Customer/Card/CustomerDescriptor.vue
  • src/pages/Customer/Card/CustomerDescriptorMenu.vue
  • src/pages/Customer/Card/CustomerSummary.vue
  • src/pages/Customer/CustomerList.vue
  • src/pages/Customer/components/CustomerSummaryTable.vue
  • src/pages/Ticket/Card/TicketCreateRequest.vue
  • src/pages/Ticket/TicketList.vue
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
3 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#790
No description provided.