feat: refs #7322 add address selection for ticket transfer #1224

Merged
jtubau merged 5 commits from 7322-addSelectAddressOnTicketTransfer into dev 2025-01-24 14:48:06 +00:00
Member
PR BACK: https://gitea.verdnatura.es/verdnatura/salix/pulls/3363
jtubau added 1 commit 2025-01-15 13:17:27 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
36473d979b
feat: refs #7322 add address selection for ticket transfer
jtubau requested review from jgallego 2025-01-15 13:22:11 +00:00
jgallego requested changes 2025-01-16 06:24:45 +00:00
Dismissed
@ -55,0 +77,4 @@
selectedClient.value = client;
};
const fetchAddresses = async (clientId) => {
Owner

@jsegarra esta estructura ya la tenemos con esta 3 veces, y necesitaria su test, pero serian 3 test idénticos, como hacemos para tener esta funcion disponible en el proyecto en algun sitio comun solo testeada una vez? (que por cierto ahora no esta ninguna)

@jsegarra esta estructura ya la tenemos con esta 3 veces, y necesitaria su test, pero serian 3 test idénticos, como hacemos para tener esta funcion disponible en el proyecto en algun sitio comun solo testeada una vez? (que por cierto ahora no esta ninguna)
Member

La idea seria crear un composable en el directorio de customer (src/pages/Customer/composables) que haga el tema de la gestion de consignatarios

La idea seria crear un composable en el directorio de customer (src/pages/Customer/composables) que haga el tema de la gestion de consignatarios
Member

Esto se consideraría refactor y lo abordaria dentro de esta tarea porque al añadir funcionalidad se genera esta situación.

Esto se consideraría refactor y lo abordaria dentro de esta tarea porque al añadir funcionalidad se genera esta situación.
jgallego marked this conversation as resolved
@ -473,0 +519,4 @@
v-model="address"
:label="t('ticketList.addressNickname')"
auto-load
:hint="!client ? t('Select a client to enable') : ''"
Owner

trabaja el proyecto con la interfaz en español para detectar que posiblement aquí falte la traduccion

trabaja el proyecto con la interfaz en español para detectar que posiblement aquí falte la traduccion
Author
Member

la traducción en español esta entre las etiquetas i18n de todas formas, es una traducción de una feat nueva que tenia que preguntarte si se autoriza su uso

la traducción en español esta entre las etiquetas i18n de todas formas, es una traducción de una feat nueva que tenia que preguntarte si se autoriza su uso
jgallego marked this conversation as resolved
jtubau reviewed 2025-01-16 09:03:41 +00:00
@ -55,0 +61,4 @@
await fetchAddresses(clientId);
};
const fetchClient = async (clientId) => {
Author
Member

Misma solución que para fetchAddresses

Revisar también el arrow function fetchAvailableAgencies que se repite en 3 ocasiones (TicketCreate, TicketCreateDialog y TicketList)

Misma solución que para fetchAddresses Revisar también el arrow function fetchAvailableAgencies que se repite en 3 ocasiones (TicketCreate, TicketCreateDialog y TicketList)
jgallego marked this conversation as resolved
jtubau requested review from jgallego 2025-01-23 07:09:52 +00:00
jgallego approved these changes 2025-01-23 09:11:58 +00:00
Dismissed
jtubau added 1 commit 2025-01-23 13:28:27 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
99df3c5df2
refactor: refs #7322 extract repeated functions and create tests
jtubau dismissed jgallego’s review 2025-01-23 13:28:27 +00:00
Reason:

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

jtubau added 1 commit 2025-01-23 13:44:01 +00:00
jtubau requested review from jgallego 2025-01-23 14:01:32 +00:00
jgallego approved these changes 2025-01-23 16:09:54 +00:00
jtubau added 1 commit 2025-01-24 07:20:34 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
878845627a
Merge branch 'dev' into 7322-addSelectAddressOnTicketTransfer
jtubau added 1 commit 2025-01-24 14:47:16 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
023a3b541a
Merge branch 'dev' into 7322-addSelectAddressOnTicketTransfer
jtubau merged commit 22ed652b9e into dev 2025-01-24 14:48:06 +00:00
jtubau deleted branch 7322-addSelectAddressOnTicketTransfer 2025-01-24 14:48:06 +00:00
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#1224
No description provided.