#7356 - Ticket module improves #1405

Merged
jsegarra merged 42 commits from warmfix_ticketList into test 2025-03-03 13:37:03 +00:00
Member
  1. Se añade la funcionalidad de que si hay un cliente seleccionado aparezca en el dialogo de crear
  2. Traducción al abonar todo con almacén: "refundTicketCreated", y la notificación de color negro debe ser verde.
  3. Campos requeridos en TicketRequest
  4. Mejoras en formulario de crear ticket para recoger si hay id del cliente o no. En Cesta también, son primos-hermanos.
  5. Corrección visual a la hora de seleccionar un consignatario
1. Se añade la funcionalidad de que si hay un cliente seleccionado aparezca en el dialogo de crear 3. Traducción al abonar todo con almacén: "refundTicketCreated", y la notificación de color negro debe ser verde. 4. Campos requeridos en TicketRequest 5. Mejoras en formulario de crear ticket para recoger si hay id del cliente o no. En Cesta también, son primos-hermanos. 5. Corrección visual a la hora de seleccionar un consignatario
jsegarra added 4 commits 2025-02-15 23:47:53 +00:00
jsegarra reviewed 2025-02-15 23:48:21 +00:00
@ -5,3 +5,3 @@
const filter = {
..._filter,
fields: ['nickname', 'street', 'city', 'id'],
fields: ['nickname', 'street', 'city', 'id', 'isActive'],
Author
Member

Se corrige un problema en un refactor de hace tiempo

Se corrige un problema en un refactor de hace tiempo
jsegarra added 1 commit 2025-02-15 23:50:41 +00:00
gitea/salix-front/pipeline/pr-test There was a failure building this commit Details
ab3ac4fdeb
fix: remove bad code
Author
Member

image

![image](/attachments/374d8d48-dd7a-463c-bf7b-02d5c567c3cb)
jsegarra added 1 commit 2025-02-15 23:52:38 +00:00
gitea/salix-front/pipeline/pr-test This commit looks good Details
1972e921df
test: fix getAddresses
jsegarra reviewed 2025-02-15 23:53:23 +00:00
@ -69,6 +69,10 @@ const props = defineProps({
type: Boolean,
default: true,
},
excludeParams: {
Author
Member

@jorgep Al final no hice una bandera sino una prop mas que luego se gestiona tambien desde useArrayData

@jorgep Al final no hice una bandera sino una prop mas que luego se gestiona tambien desde useArrayData
jorgep marked this conversation as resolved
jsegarra requested review from jgallego 2025-02-15 23:53:39 +00:00
jsegarra added 1 commit 2025-02-16 02:18:17 +00:00
gitea/salix-front/pipeline/pr-test This commit looks good Details
2ec5c2b49f
fix: ticketList columnfilter
jsegarra added 2 commits 2025-02-16 22:46:09 +00:00
Author
Member

image
image

![image](/attachments/1fa31085-bf32-4784-a371-7dfd069682d9) ![image](/attachments/ccb4fb58-4f47-4fd3-a859-d0a4a4d5a174)
jsegarra added 1 commit 2025-02-16 22:47:35 +00:00
gitea/salix-front/pipeline/pr-test This commit looks good Details
3e3713a937
perf: remove unnussed import
jgallego requested review from alexm 2025-02-17 06:25:56 +00:00
jgallego approved these changes 2025-02-17 06:26:01 +00:00
Dismissed
jsegarra added 1 commit 2025-02-17 08:52:32 +00:00
gitea/salix-front/pipeline/pr-test This commit looks good Details
c212b8e21c
Merge branch 'test' into warmfix_ticketList
jsegarra requested review from jorgep 2025-02-17 11:20:47 +00:00
Member

En test funcionan mal las fechas de ticketList?

En test funcionan mal las fechas de ticketList?
jorgep reviewed 2025-02-17 15:32:24 +00:00
@ -1,5 +1,5 @@
<script setup>
import { ref, computed } from 'vue';
import { ref, computed, provide, inject, onMounted } from 'vue';
Member

Quitar provide.

Quitar provide.
jsegarra marked this conversation as resolved
jorgep reviewed 2025-02-17 15:33:57 +00:00
@ -88,2 +91,3 @@
const searchbar = ref(null);
defineExpose({ search, params: userParams, remove });
onMounted(() => {
Member

Debajo de la variable isLoading, quedamos que todas las funciones agrupadas.

Debajo de la variable isLoading, quedamos que todas las funciones agrupadas.
jsegarra marked this conversation as resolved
jorgep reviewed 2025-02-17 15:35:21 +00:00
@ -93,1 +99,3 @@
if (evt && $props.disableSubmitEvent) return;
if ($props.useSearchbar) {
if (!searchbar.value) {
console.error('Searchbar not found');
Member

@alexm dejamos console.error? Entiendo que esto solo sirve para depurar nosotros.

@alexm dejamos console.error? Entiendo que esto solo sirve para depurar nosotros.
jsegarra marked this conversation as resolved
jorgep reviewed 2025-02-17 15:36:19 +00:00
@ -94,0 +110,4 @@
}
}
}
if (evt && $props.disableSubmitEvent) debugger;
Member

Quitar.

Quitar.
jsegarra marked this conversation as resolved
jorgep reviewed 2025-02-17 15:39:23 +00:00
@ -94,0 +104,4 @@
if (typeof $props.useSearchbar === 'function') {
$props.useSearchbar(userParams.value);
if (Object.keys(userParams.value).length == 0) {
Member

!(Object.keys(userParams.value).length

**!(Object.keys(userParams.value).length**
jsegarra marked this conversation as resolved
jorgep reviewed 2025-02-17 15:47:50 +00:00
@ -75,3 +75,3 @@
}
async function fetch({ append = false, updateRouter = true }) {
async function fetch(fetchOptions) {
Member

fetchOptions = {}; Si a alguien llama a fetch sin {} , va a fallar el desestructuring

fetchOptions = {}; Si a alguien llama a fetch sin {} , va a fallar el desestructuring
Author
Member

Si alguien llama a fetch sin {}, entrará en juego la siguiente linea:78
let { append = false, updateRouter = true } = fetchOptions;
y como luego tiene ?., no fallará

correcto?

Si alguien llama a fetch sin {}, entrará en juego la siguiente linea:78 ` let { append = false, updateRouter = true } = fetchOptions; ` y como luego tiene ?., no fallará correcto?
Member

Te adjunto una captura. Lo que dice de que luego ?. está claro, el fallo está en el desestructuring.

Te adjunto una captura. Lo que dice de que luego ?. está claro, el fallo está en el desestructuring.
Author
Member

fetchOptions ?? {}

fetchOptions ?? {}
Member

vale

vale
jorgep marked this conversation as resolved
jorgep reviewed 2025-02-17 15:58:00 +00:00
@ -94,0 +102,4 @@
return;
}
if (typeof $props.useSearchbar === 'function') {
$props.useSearchbar(userParams.value);
Member

A nivel de lógica lo veo bien, pero separaría la lógica de la fn con la del booleano. o pasas 2 props o pasas un obj ,use-searchbar suena a que vas a usar searchbar, pero desde ticketList estás pasando un fn previa de comprobación. No me parece correcto el nombre.

Yo haría algo así:

searchbar: true -> solo usa la fn de searchbar. searchbar: { validateFn: myFn }.

@alexm que opinas?

A nivel de lógica lo veo bien, pero separaría la lógica de la fn con la del booleano. o pasas 2 props o pasas un obj ,use-searchbar suena a que vas a usar searchbar, pero desde ticketList estás pasando un fn previa de comprobación. No me parece correcto el nombre. Yo haría algo así: searchbar: true -> solo usa la fn de searchbar. searchbar: { validateFn: myFn }. @alexm que opinas?
Author
Member

Yo le estoy pasando un callback para que se ejecute, como en FormModel.saveFn porque que llame o no a la searchbar depende de un criterio.
Si esa prop, le digo booleano, es que siempre la quiero. Ahora por defecto es false
Pasarle 2 props para una logica? Yo eso si que no lo veo
Sugerencia de nombre?

Yo le estoy pasando un callback para que se ejecute, como en FormModel.saveFn porque que llame o no a la searchbar depende de un criterio. Si esa prop, le digo booleano, es que siempre la quiero. Ahora por defecto es false Pasarle 2 props para una logica? Yo eso si que no lo veo Sugerencia de nombre?
Member

searchbar-handler o algo así. Yo intentaría que quede claro que tu quieres usar un función de validación previa, por eso te había propuesto usar un objeto, pero que diga @alexm.

searchbar-handler o algo así. Yo intentaría que quede claro que tu quieres usar un función de validación previa, por eso te había propuesto usar un objeto, pero que diga @alexm.
jorgep reviewed 2025-02-17 16:00:51 +00:00
@ -1,5 +1,5 @@
<script setup>
import { ref } from 'vue';
import { computed, ref } from 'vue';
Member

quitar computed

quitar computed
jsegarra marked this conversation as resolved
jorgep reviewed 2025-02-17 16:01:24 +00:00
@ -8,6 +8,8 @@ import VnInput from 'src/components/common/VnInput.vue';
import VnInputDate from 'components/common/VnInputDate.vue';
import VnSelect from 'src/components/common/VnSelect.vue';
import VnSelectWorker from 'src/components/common/VnSelectWorker.vue';
import { Notify } from 'quasar';
Member

Quitar

Quitar
Author
Member

no me salia para remover
Gracias

no me salia para remover Gracias
jsegarra marked this conversation as resolved
jorgep reviewed 2025-02-17 16:09:34 +00:00
jorgep requested changes 2025-02-17 16:10:17 +00:00
Dismissed
@ -366,2 +366,4 @@
cy.get('.q-btn').contains(buttonText).click();
});
Cypress.Commands.add('selectOptionBeta', (index = 1) => {
cy.get(`[role="listbox"] .q-item:nth-child(${index})`).click();
Member

Lo usas en algún sitio?

Lo usas en algún sitio?
Author
Member

ticketList.spec.js:63

ticketList.spec.js:63
Author
Member

he visto que este path se usaba en mas test, así que he cogido el nombre y el textoy lo he movido a commands.
He usado ese command en mas ocurrencias

he visto que este path se usaba en mas test, así que he cogido el nombre y el textoy lo he movido a commands. He usado ese command en mas ocurrencias
jorgep marked this conversation as resolved
Author
Member

En test funcionan mal las fechas de ticketList?

como?
Test e2e ticketFilter o ticketList?

> En test funcionan mal las fechas de ticketList? como? Test e2e ticketFilter o ticketList?
jsegarra added 1 commit 2025-02-17 18:33:01 +00:00
gitea/salix-front/pipeline/pr-test There was a failure building this commit Details
9b1d1247b1
Merge branch 'test' into warmfix_ticketList
jsegarra added 2 commits 2025-02-17 18:33:35 +00:00
jsegarra dismissed jgallego’s review 2025-02-17 18:33:35 +00:00
Reason:

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

jsegarra added 1 commit 2025-02-17 21:57:16 +00:00
gitea/salix-front/pipeline/pr-test This commit looks good Details
7f370dc29c
test: improve getOption command
jsegarra added 2 commits 2025-02-17 22:27:26 +00:00
gitea/salix-front/pipeline/pr-test There was a failure building this commit Details
e6e21b61bd
perf: orderList
jsegarra added 1 commit 2025-02-17 23:45:55 +00:00
gitea/salix-front/pipeline/pr-test This commit looks good Details
3ca73d03a0
test: fix
jsegarra requested review from jorgep 2025-02-17 23:46:02 +00:00
jsegarra added 1 commit 2025-02-18 08:31:41 +00:00
gitea/salix-front/pipeline/pr-test This commit looks good Details
93a0017339
Merge branch 'test' into warmfix_ticketList
jorgep requested changes 2025-02-18 10:38:10 +00:00
Dismissed
@ -93,0 +103,4 @@
if (!searchbar.value) {
return;
}
if (typeof $props.useSearchbar === 'function') {
Member

Tras la conversación que hemos tenido quedamos en usar options.

Tras la conversación que hemos tenido quedamos en usar options.
jsegarra added 4 commits 2025-02-18 23:05:31 +00:00
jsegarra added 1 commit 2025-02-18 23:12:06 +00:00
gitea/salix-front/pipeline/pr-test This commit looks good Details
f33d396d82
perf: minor changes
jsegarra requested review from jorgep 2025-02-18 23:13:13 +00:00
Member

@jsegarra
image

Tienes que poner fetchOptions = {} para que prevenir errores.

@jsegarra ![image](/attachments/d50ff017-d6d7-41c6-8722-5c6f01b158a9) Tienes que poner fetchOptions = {} para que prevenir errores.
jorgep reviewed 2025-02-19 08:38:45 +00:00
@ -0,0 +40,4 @@
cy.location('href').should('contain', '#/ticket/999999');
});
});
function today() {
Member

No sé si lo tenemos en nuestra librería de utilidades, pero me parece interesante ponerla(usando Date.vnNew()). Solo es comentario

No sé si lo tenemos en nuestra librería de utilidades, pero me parece interesante ponerla(usando Date.vnNew()). Solo es comentario
Author
Member

creamos tarea

creamos tarea
jorgep marked this conversation as resolved
jorgep requested changes 2025-02-19 08:40:35 +00:00
Dismissed
jorgep left a comment
Member

Revisar fetchOptions fn fetch, prevenir error.

Revisar fetchOptions fn fetch, prevenir error.
jsegarra added 2 commits 2025-02-19 11:20:03 +00:00
jsegarra requested review from jorgep 2025-02-19 11:21:41 +00:00
jorgep approved these changes 2025-02-19 14:56:15 +00:00
Dismissed
jsegarra added 1 commit 2025-02-21 07:38:18 +00:00
gitea/salix-front/pipeline/pr-test This commit looks good Details
57c0171bdd
fix: transfer style
jsegarra dismissed jorgep’s review 2025-02-21 07:38:18 +00:00
Reason:

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

alexm requested changes 2025-02-21 08:06:25 +00:00
Dismissed
alexm left a comment
Member

No acabo de ver la funcionalidad de filtrado. VnFilterPanel y VnSearchbar por si solos pueden filtrar y pueden acceder a los userParams.
No entiendo el pq debe utilizar vnFilterPanel el filtrado de la searchbar

No acabo de ver la funcionalidad de filtrado. VnFilterPanel y VnSearchbar por si solos pueden filtrar y pueden acceder a los userParams. No entiendo el pq debe utilizar vnFilterPanel el filtrado de la searchbar
Author
Member

No acabo de ver la funcionalidad de filtrado. VnFilterPanel y VnSearchbar por si solos pueden filtrar y pueden acceder a los userParams.
No entiendo el pq debe utilizar vnFilterPanel el filtrado de la searchbar

El problema que tienen es que si buscan un cliente les pone un rango de fechas y eso está mal, tienen que quitar las chips
Lo que quieren es que al cargar la vista de ticketList, poner el id de cliente sin fechas, como estaba en Salix.

> No acabo de ver la funcionalidad de filtrado. VnFilterPanel y VnSearchbar por si solos pueden filtrar y pueden acceder a los userParams. > No entiendo el pq debe utilizar vnFilterPanel el filtrado de la searchbar El problema que tienen es que si buscan un cliente les pone un rango de fechas y eso está mal, tienen que quitar las chips Lo que quieren es que al cargar la vista de ticketList, poner el id de cliente sin fechas, como estaba en Salix.
jsegarra added 1 commit 2025-02-21 08:26:54 +00:00
gitea/salix-front/pipeline/pr-test There was a failure building this commit Details
fdf9cfab74
Merge branch 'test' into warmfix_ticketList
Member

No acabo de ver la funcionalidad de filtrado. VnFilterPanel y VnSearchbar por si solos pueden filtrar y pueden acceder a los userParams.
No entiendo el pq debe utilizar vnFilterPanel el filtrado de la searchbar

El problema que tienen es que si buscan un cliente les pone un rango de fechas y eso está mal, tienen que quitar las chips
Lo que quieren es que al cargar la vista de ticketList, poner el id de cliente sin fechas, como estaba en Salix.

Si el problema lo tiene vnFilterPanel, se puede meter ahi la logica, no entiendo el usar la searchbar

> > No acabo de ver la funcionalidad de filtrado. VnFilterPanel y VnSearchbar por si solos pueden filtrar y pueden acceder a los userParams. > > No entiendo el pq debe utilizar vnFilterPanel el filtrado de la searchbar > > El problema que tienen es que si buscan un cliente les pone un rango de fechas y eso está mal, tienen que quitar las chips > Lo que quieren es que al cargar la vista de ticketList, poner el id de cliente sin fechas, como estaba en Salix. > > Si el problema lo tiene vnFilterPanel, se puede meter ahi la logica, no entiendo el usar la searchbar
jsegarra added 5 commits 2025-02-23 13:08:34 +00:00
jsegarra added 1 commit 2025-02-23 19:57:27 +00:00
gitea/salix-front/pipeline/pr-test There was a failure building this commit Details
ed43f413f5
perf: remove comments
jsegarra changed title from Warmfix TicketList to #7356 - Warmfix TicketList 2025-02-23 21:02:43 +00:00
jsegarra requested review from alexm 2025-02-23 21:04:37 +00:00
jsegarra added 2 commits 2025-02-24 01:41:28 +00:00
gitea/salix-front/pipeline/pr-test This commit looks good Details
43bbf05adf
perf: apply search
alexm requested changes 2025-02-24 06:20:05 +00:00
Dismissed
@ -95,3 +116,3 @@
store.filter.where = {};
isLoading.value = true;
const filter = { ...userParams.value, ...$props.modelValue };
const filter = { ...userParams.value, ...$props.modelValue, ...evt };
Member

El evento en el filtro?

El evento en el filtro?
Author
Member

Si, porque ocurría que al buscar un ticket, el param search no se pasaba

Si, porque ocurría que al buscar un ticket, el param search no se pasaba
@ -117,3 +127,3 @@
let filter = { params: { search: searchText.value } };
if (filterPanel?.value?.filterPanelRef) {
Member

Pq la searchbar tiene que ejecutar el search del filterpanel?

Pq la searchbar tiene que ejecutar el search del filterpanel?
Author
Member

Si queremos evitar tener la lógica duplicada, lo interesante seria ver técnicas para llamar a un componente desde otro.
En la solicitud anterior se hacia un provide/ineject que era mas limpio, aquí se tiene que hacer spaguetti-code porque hay que pasarle el filtro como prop, ya que el el orden en el que se montan los componentes no permite detectar el VnFilterPanel desde la searchbar

Ejecutando el search del filterpanel definimos solo un excludeParams

Si queremos evitar tener la lógica duplicada, lo interesante seria ver técnicas para llamar a un componente desde otro. En la solicitud anterior se hacia un provide/ineject que era mas limpio, aquí se tiene que hacer spaguetti-code porque hay que pasarle el filtro como prop, ya que el el orden en el que se montan los componentes no permite detectar el VnFilterPanel desde la searchbar Ejecutando el search del filterpanel definimos solo un excludeParams
alexm requested changes 2025-02-24 06:21:36 +00:00
Dismissed
alexm left a comment
Member

La logica de excluceParams y exclude no la acabo de ver

La logica de excluceParams y exclude no la acabo de ver
@ -451,0 +482,4 @@
<TicketFilter
ref="filterPanelRef"
data-key="TicketList"
:excludeParams="{ ...userParams }"
Member

exlcudeParams son todos los userParams?

exlcudeParams son todos los userParams?
Author
Member

excludePArams, son los parametros que no tienen que añadirse a la URL
En Salix se llamana excludedParams, le sobra una d, pero sin ella creo que se entiende mejor

excludePArams, son los parametros que no tienen que añadirse a la URL En Salix se llamana excludedParams, le sobra una d, pero sin ella creo que se entiende mejor
jsegarra requested review from alexm 2025-02-24 07:22:40 +00:00
jsegarra changed title from #7356 - Warmfix TicketList to Warmfix #7356 - Ticket module 2025-03-01 01:40:55 +00:00
jsegarra added 2 commits 2025-03-02 21:43:10 +00:00
jsegarra added 2 commits 2025-03-02 22:22:27 +00:00
jsegarra changed title from Warmfix #7356 - Ticket module to #7356 - Ticket module improves 2025-03-02 22:23:13 +00:00
Author
Member

He trasladado la lógica del filtrado a otra PR #1530
De esta manera separamos lo que es la búsqueda y filtrado de cuestiones visuales y de UI/UX

He trasladado la lógica del filtrado a otra PR https://gitea.verdnatura.es/verdnatura/salix-front/pulls/1530 De esta manera separamos lo que es la búsqueda y filtrado de cuestiones visuales y de UI/UX
alexm requested changes 2025-03-03 06:54:34 +00:00
Dismissed
@ -0,0 +11,4 @@
cy.waitForElement('.q-page');
cy.intercept('GET', /\/api\/Tickets\/filter/).as('ticketFilter');
cy.searchBtnFilterPanel();
cy.waitRequest('@ticketFilter', ({ request }) => {
Member

No acabo de ver la funcion cy.waitRequest.
Usando lo nativo mejor el codigo

cy.wait('@ticketFilter').then({ request }) => {
...
})
No acabo de ver la funcion cy.waitRequest. Usando lo nativo mejor el codigo ``` cy.wait('@ticketFilter').then({ request }) => { ... }) ```
Author
Member

el sentido de este comando es delegar la lógica de interceptar peticiones desde un sitio en vez de que cada uno pueda hacerlo variable.
Por unificar, vamos, ya que al fin y al cabo, le pasas el alias y el cb

el sentido de este comando es delegar la lógica de interceptar peticiones desde un sitio en vez de que cada uno pueda hacerlo variable. Por unificar, vamos, ya que al fin y al cabo, le pasas el alias y el cb
Member

Sigo sin ver un commnado personalizado, que con codigo nativo se lee mas claro

Sigo sin ver un commnado personalizado, que con codigo nativo se lee mas claro
jsegarra added 1 commit 2025-03-03 08:55:04 +00:00
gitea/salix-front/pipeline/pr-test This commit looks good Details
d041bf6dc1
Merge branch 'test' into warmfix_ticketList
jsegarra requested review from alexm 2025-03-03 09:43:33 +00:00
jgallego approved these changes 2025-03-03 12:18:48 +00:00
alexm approved these changes 2025-03-03 12:58:50 +00:00
alexm added 1 commit 2025-03-03 13:10:47 +00:00
gitea/salix-front/pipeline/pr-test This commit looks good Details
b325ce191d
Merge branch 'test' into warmfix_ticketList
jsegarra scheduled this pull request to auto merge when all checks succeed 2025-03-03 13:21:41 +00:00
jsegarra merged commit ea50caff3f into test 2025-03-03 13:37:03 +00:00
alexm deleted branch warmfix_ticketList 2025-03-03 13:40:35 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
4 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#1405
No description provided.