verdnatura/salix-front
verdnatura
/
salix-front
2
2
Fork 1
Code Issues Pull Requests 95 Releases Wiki Activity

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

Open
jtubau wants to merge 1 commits from 7322-addSelectAddressOnTicketTransfer into dev
pull from: 7322-addSelectAddressOnTicketTransfer
merge into: verdnatura:dev
Conversation 0 Commits 1 Files Changed 1 +73 -10
jtubau commented 2025-01-15 13:17:26 +00:00
Member
Copy Link

PR BACK: verdnatura/salix#3363

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
src/pages/Ticket/Card/TicketDescriptorMenu.vue
@ -55,0 +77,4 @@
selectedClient.value = client;
};
const fetchAddresses = async (clientId) => {
jgallego commented 2025-01-16 06:24:43 +00:00
Owner
Copy Link

@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)
jsegarra commented 2025-01-16 06:55:51 +00:00
Member
Copy Link

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
jsegarra commented 2025-01-16 06:57:12 +00:00
Member
Copy Link

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.
src/pages/Ticket/Card/TicketDescriptorMenu.vue
@ -473,0 +519,4 @@
v-model="address"
:label="t('ticketList.addressNickname')"
auto-load
:hint="!client ? t('Select a client to enable') : ''"
jgallego commented 2025-01-16 06:18:08 +00:00
Owner
Copy Link

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
jtubau commented 2025-01-16 09:11:52 +00:00
Author
Member
Copy Link

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
jtubau reviewed 2025-01-16 09:03:41 +00:00
src/pages/Ticket/Card/TicketDescriptorMenu.vue
@ -55,0 +61,4 @@
await fetchAddresses(clientId);
};
const fetchClient = async (clientId) => {
jtubau commented 2025-01-16 09:03:41 +00:00
Author
Member
Copy Link

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)
All checks were successful
gitea/salix-front/pipeline/pr-dev This commit looks good
Required
Details
This pull request doesn't have enough approvals yet. 0 of 1 approvals granted.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
Sign in to join this conversation.
Reviewers
No reviewers
jgallego
Labels
Clear labels   
bug

Something is not working

  
CR / Tests passed

CR / Tests passed

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
invalid

Something is wrong

  
Missing Tests

Missing Tests

  
wontfix

This won't be fixed

No Label
bug
CR / Tests passed
duplicate
enhancement
invalid
Missing Tests
wontfix
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
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.
Delete Branch "7322-addSelectAddressOnTicketTransfer"

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?