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

Ticket basic data #474

Merged
jsegarra merged 15 commits from :feature/TicketBasicData into dev 2024-06-25 20:28:01 +00:00
Conversation 0 Commits 15 Files Changed 9 +1011 -6
hyervoni commented 2024-06-21 19:57:20 +00:00
Contributor
Copy Link
No description provided.
hyervoni added 5 commits 2024-06-21 19:57:21 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
3d9d41afcb Add deep true
wbuezas added 1 commit 2024-06-21 19:58:00 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
3d0cbcfd12 remove form model expose
hyervoni requested review from alexm 2024-06-21 20:00:03 +00:00
hyervoni requested review from jsegarra 2024-06-21 20:00:03 +00:00
jsegarra reviewed 2024-06-25 07:11:09 +00:00
src/pages/Ticket/Card/BasicData/TicketBasicDataForm.vue Outdated
@ -0,0 +36,4 @@
const agenciesOptions = ref([]);
const zonesOptions = ref([]);
const addresses = ref([]);
const _formData = ref($props.formData);
jsegarra commented 2024-06-25 07:11:09 +00:00
Member
Copy Link

duda, porque _formData y no formData??

duda, porque _formData y no formData??
jsegarra requested changes 2024-06-25 07:25:01 +00:00
Dismissed
src/pages/Ticket/Card/BasicData/BasicDataTable.vue
@ -0,0 +55,4 @@
name: 'description',
align: 'left',
},
{
jsegarra commented 2024-06-25 07:21:40 +00:00
Member
Copy Link

esta columna tiene que ser condicional en base al campo priceDifferences.haveDifferences

esta columna tiene que ser condicional en base al campo priceDifferences.haveDifferences
jsegarra marked this conversation as resolved
src/pages/Ticket/Card/BasicData/BasicDataTable.vue
@ -0,0 +105,4 @@
);
});
const totalDifference = computed(() => {
jsegarra commented 2024-06-25 07:16:19 +00:00
Member
Copy Link

revisar porque se muestra un resultado diferente

revisar porque se muestra un resultado diferente
jsegarra commented 2024-06-25 07:18:16 +00:00
Member
Copy Link

Hay que eliminar * item.quantity

Hay que eliminar * item.quantity
jsegarra commented 2024-06-25 07:23:58 +00:00
Member
Copy Link

El cambio anterior, con el registro 31 ✅, pero con el 7 ❌

El cambio anterior, con el registro 31 ✅, pero con el 7 ❌
wbuezas commented 2024-06-25 11:35:50 +00:00
Contributor
Copy Link

Fixed.

Commit: 266657fab5

Fixed. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/266657fab5bd31f330b4a18bfbbf66dc238bf5a5
jsegarra marked this conversation as resolved
src/pages/Ticket/Card/BasicData/TicketBasicDataForm.vue Outdated
@ -0,0 +396,4 @@
</QIcon>
</template>
</VnSelect>
<VnInput :label="t('basicData.alias')" v-model="_formData.nickname" />
jsegarra commented 2024-06-25 07:12:36 +00:00
Member
Copy Link

deberia ser requerido no? Quiero decir, le falta el *

deberia ser requerido no? Quiero decir, le falta el *
jsegarra commented 2024-06-25 08:35:14 +00:00
Member
Copy Link

Revisar el resto de campos

Revisar el resto de campos
wbuezas commented 2024-06-25 11:44:46 +00:00
Contributor
Copy Link

Añadido.

Commit: 54e3eb6f0d

Añadido. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/54e3eb6f0d7de3726ffa6269e7d9c02b2b9faa5d
src/pages/Ticket/Card/BasicData/TicketBasicDataForm.vue
@ -0,0 +420,4 @@
:required="true"
@focus="agencyFetchRef.fetch()"
/>
<VnSelect
jsegarra commented 2024-06-25 07:12:04 +00:00
Member
Copy Link

No se ve el texto, sino el id del registro

No se ve el texto, sino el id del registro
wbuezas commented 2024-06-25 14:34:08 +00:00
Contributor
Copy Link

No se ve el name ya que no hay no existen zoneOptions para los registros cargados, es algo medio raro la verdad como esta planteado, ya que en salix si desplegas el input de zonas te va a aparecer que no hay resultados.
Y en lilium esto es un problema porque el QSelect utiliza las options para mapear el resultado en base al value.

No se ve el name ya que no hay no existen `zoneOptions` para los registros cargados, es algo medio raro la verdad como esta planteado, ya que en salix si desplegas el input de zonas te va a aparecer que no hay resultados. Y en lilium esto es un problema porque el `QSelect` utiliza las options para mapear el resultado en base al value.
src/pages/Ticket/Card/BasicData/TicketBasicDataView.vue
@ -0,0 +152,4 @@
onBeforeMount(async () => await getTicketData());
</script>
<template>
<QStepper
jsegarra commented 2024-06-25 07:24:59 +00:00
Member
Copy Link

Revisamos el stepper porque:

  1. Propuesta de mejora para colocar los botones
  2. si pulso sobre siguiente se llama a priceDifferences, pero si pulso en diferencia de precio ❌
Revisamos el stepper porque: 1. Propuesta de mejora para colocar los botones 2. si pulso sobre siguiente se llama a priceDifferences, pero si pulso en diferencia de precio ❌
wbuezas commented 2024-06-25 12:20:26 +00:00
Contributor
Copy Link

Mejora de organización en los botones y removí la navegación vía header para no generar inconvenientes, porque probablemente se necesite refactorizar algo para implementarlo.

Commit: 4de347d0e8

Mejora de organización en los botones y removí la navegación vía header para no generar inconvenientes, porque probablemente se necesite refactorizar algo para implementarlo. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/4de347d0e8f180a05035282ddf863fd8d5b3918b
jsegarra requested changes 2024-06-25 07:31:46 +00:00
Dismissed
src/pages/Ticket/Card/BasicData/BasicDataTable.vue
@ -0,0 +184,4 @@
</span>
</QCardSection>
<QCardSection
v-for="(action, index) in ticketUpdateActions"
jsegarra commented 2024-06-25 07:31:45 +00:00
Member
Copy Link

Creo que falta añadir una condición de show según el valor totalPriceDifference
Idem, con el registro 31 ✅ y con el 7 ❌

Creo que falta añadir una condición de show según el valor totalPriceDifference Idem, con el registro 31 ✅ y con el 7 ❌
wbuezas added 1 commit 2024-06-25 11:35:20 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
266657fab5 Fix total difference
jsegarra added 3 commits 2024-06-25 11:35:56 +00:00
wbuezas added 2 commits 2024-06-25 11:43:19 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
ba8145eabb Add required rule to VnInputTime and VnInputDate
wbuezas added 1 commit 2024-06-25 12:19:21 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
4de347d0e8 Rmove nav navigation and organize buttons
alexm approved these changes 2024-06-25 12:21:07 +00:00
Dismissed
jsegarra added 2 commits 2024-06-25 20:27:23 +00:00
jsegarra dismissed alexm’s review 2024-06-25 20:27:23 +00:00
Reason:

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

jsegarra approved these changes 2024-06-25 20:27:30 +00:00
jsegarra scheduled this pull request to auto merge when all checks succeed 2024-06-25 20:27:37 +00:00
jsegarra merged commit d60ddfac6f into dev 2024-06-25 20:28:01 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
jsegarra
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
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#474
No description provided.
Delete Branch ":feature/TicketBasicData"

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?