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

CambiosSolicitadosEntries #201

Merged
alexm merged 17 commits from :CambiosSolicitadosEntries into dev 2024-03-14 06:39:45 +00:00
Conversation 1 Commits 17 Files Changed 8 +112 -64
hyervoni commented 2024-02-23 15:01:28 +00:00
Contributor
Copy Link
No description provided.
hyervoni added 8 commits 2024-02-23 15:01:28 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
3e04c86000 reduce EntrySummary distance between table rows
hyervoni requested review from jgallego 2024-02-23 15:05:00 +00:00
hyervoni requested review from alexm 2024-02-23 15:05:01 +00:00
hyervoni requested review from jsegarra 2024-02-23 15:05:01 +00:00
wbuezas commented 2024-02-23 15:09:42 +00:00
Contributor
Copy Link

Consideraciones:

  • Utilicé el icono que se encuentra actualmente en lilium que representa redada, pero el que esta agregado parece que no es exactamente igual que en salix, @jsegarra me comento que lo iban a revisar con el muchacho que agregó los iconos
  • Se aplicó el label "Sin resultados" a las tablas que no contienen resultados, pero hay otros componentes que deberian mostrarlo como VnPaginate y VnLog
**Consideraciones:** - Utilicé el icono que se encuentra actualmente en lilium que representa redada, pero el que esta agregado parece que no es exactamente igual que en salix, @jsegarra me comento que lo iban a revisar con el muchacho que agregó los iconos - Se aplicó el label "Sin resultados" a las tablas que no contienen resultados, pero hay otros componentes que deberian mostrarlo como `VnPaginate` y `VnLog`
jsegarra requested changes 2024-02-23 22:28:32 +00:00
src/pages/Entry/Card/EntryBuys.vue Outdated
@ -398,3 +421,3 @@
<!-- Esta última row es utilizada para agregar un espaciado y así marcar una diferencia visual entre los diferentes buys -->
<QTr v-if="props.rowIndex !== rows.length - 1" class="separation-row">
<QTd colspan="12" style="height: 24px" />
<QTd colspan="12" style="height: 16px" />
jsegarra commented 2024-02-23 22:20:00 +00:00
Member
Copy Link

Este height es único de entrybuys?

Este height es único de entrybuys?
wbuezas commented 2024-02-26 11:47:05 +00:00
Contributor
Copy Link

Variable global aplicada

Commit: 90bfba0290

Variable global aplicada Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/90bfba02909a198a89e25e8cabeb8f87751bc78e
jsegarra marked this conversation as resolved
src/pages/Entry/Card/EntryDescriptor.vue Outdated
@ -93,2 +98,4 @@
};
const refetchEntryDescriptorData = async () => {
await entryDescriptorRef.value.getData();
jsegarra commented 2024-02-23 22:26:01 +00:00
Member
Copy Link

Yo movería la línea si refetch solo hace eso

Yo movería la línea si refetch solo hace eso
wbuezas commented 2024-02-26 13:28:22 +00:00
Contributor
Copy Link

Quedo solucionado en el siguiente commit: 740eedbe44

Quedo solucionado en el siguiente commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/740eedbe44f9bb0d8095efb4fe89371040543807
jsegarra marked this conversation as resolved
src/pages/Entry/Card/EntryDescriptor.vue Outdated
@ -96,2 +112,4 @@
openReport(`Entries/${route.params.id}/entry-order-pdf`);
};
watch;
jsegarra commented 2024-02-23 22:21:24 +00:00
Member
Copy Link

Mmm

Mmm
👀 1
wbuezas commented 2024-02-26 13:29:15 +00:00
Contributor
Copy Link

Pude encontrar una solución 1000 veces mejor :D, y además corregí un problemita que tenía el FormModel, la modificación está relacionada a la solución que vimos el viernes en meet.

Commit: 740eedbe44

Pude encontrar una solución 1000 veces mejor :D, y además corregí un problemita que tenía el `FormModel`, la modificación está relacionada a la solución que vimos el viernes en meet. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/740eedbe44f9bb0d8095efb4fe89371040543807
src/pages/Entry/Card/EntrySummary.vue Outdated
@ -339,3 +339,3 @@
class="separation-row"
>
<QTd colspan="10" style="height: 24px" />
<QTd colspan="10" style="height: 16px" />
jsegarra commented 2024-02-23 22:28:17 +00:00
Member
Copy Link

Uff segunda ocurrencia…creo que tocará definirlo en scss

Uff segunda ocurrencia…creo que tocará definirlo en scss
wbuezas commented 2024-02-26 11:46:32 +00:00
Contributor
Copy Link

Aplicado

Commit: 90bfba0290

Aplicado Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/90bfba02909a198a89e25e8cabeb8f87751bc78e
jsegarra marked this conversation as resolved
wbuezas added 1 commit 2024-02-23 23:36:34 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
4d2dacff6f add no results label
jgallego requested changes 2024-02-24 06:58:32 +00:00
src/pages/Entry/Card/EntrySummary.vue
@ -289,9 +289,9 @@ const fetchEntryBuys = async () => {
<QTable
:rows="entryBuys"
:columns="entriesTableColumns"
jgallego commented 2024-02-24 06:56:35 +00:00
Owner
Copy Link

En esta seccion en salix hay 3 columnas/cards, en lilium debe ser igual.
Hay una pequeña modificación:
Lo que ahora hay en la card datos basicos se divide en dos columnas
card 1(Link a basic data): Comisión, Moneda, Empresa ,Referencia, Núm. factura
card 2(Link a basic data): Pedida, Confirmado, Asentado, Redada, Inventario
card 3(Link a travel): Esta seccion igual que esta son los datos de envio

En esta seccion en salix hay 3 columnas/cards, en lilium debe ser igual. Hay una pequeña modificación: Lo que ahora hay en la card datos basicos se divide en dos columnas card 1(Link a basic data): Comisión, Moneda, Empresa ,Referencia, Núm. factura card 2(Link a basic data): Pedida, Confirmado, Asentado, Redada, Inventario card 3(Link a travel): Esta seccion igual que esta son los datos de envio
wbuezas commented 2024-02-26 13:53:03 +00:00
Contributor
Copy Link

Implementado!

Commit: 84a0be0a5c

Implementado! Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/84a0be0a5c787448d1c34f459bbae4fc166fe977
jsegarra commented 2024-02-27 06:53:56 +00:00
Member
Copy Link

Resuelvo conversación porque lo he visto bien y he compartido la evidencia del cambio con Javi

Resuelvo conversación porque lo he visto bien y he compartido la evidencia del cambio con Javi
jsegarra marked this conversation as resolved
src/pages/Entry/EntryLatestBuys.vue Outdated
@ -289,3 +289,3 @@
<template #body-cell-itemFk="{ row }">
<QTd @click.stop>
<QBtn flat color="blue">
<QBtn flat color="primary">
jgallego commented 2024-02-24 06:51:20 +00:00
Owner
Copy Link

en esta seccion hay que añadir el filtro lateral de busqueda igual que esta en salix.

en esta seccion hay que añadir el filtro lateral de busqueda igual que esta en salix.
jsegarra commented 2024-02-26 07:45:14 +00:00
Member
Copy Link

https://redmine.verdnatura.es/issues/6537#note-6

https://redmine.verdnatura.es/issues/6537#note-6
wbuezas commented 2024-02-26 11:20:14 +00:00
Contributor
Copy Link

Buenas @jgallego por nuestra parte lo teniamos en visto esto en esta tarea: https://mindshore-verdnatura.atlassian.net/jira/software/projects/SCRUM/boards/1/backlog?selectedIssue=SCRUM-162&text=entry

Si te parece lo mandamos en una PR aparte

Buenas @jgallego por nuestra parte lo teniamos en visto esto en esta tarea: https://mindshore-verdnatura.atlassian.net/jira/software/projects/SCRUM/boards/1/backlog?selectedIssue=SCRUM-162&text=entry Si te parece lo mandamos en una PR aparte
jgallego commented 2024-02-26 13:17:25 +00:00
Owner
Copy Link

perfecto, resuelvo conversacion

perfecto, resuelvo conversacion
jgallego marked this conversation as resolved
wbuezas added 1 commit 2024-02-26 11:45:27 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
90bfba0290 Create global variable for table separation row
wbuezas added 1 commit 2024-02-26 12:55:14 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
740eedbe44 Improve reactivity of card descriptor
wbuezas added 1 commit 2024-02-26 13:52:33 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
84a0be0a5c Add new card to entry summary and fix links redirections
wbuezas added 1 commit 2024-02-26 14:07:39 +00:00
wbuezas added 1 commit 2024-02-26 14:28:14 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
e372c04d35 change variable name
jsegarra requested changes 2024-02-27 06:52:31 +00:00
src/pages/Entry/Card/EntrySummary.vue Outdated
@ -149,3 +149,3 @@
>
<template #header-left>
<a class="header link" :href="entryUrl">
<router-link
jsegarra commented 2024-02-27 06:52:14 +00:00
Member
Copy Link

Revisamos porque no apunta a la misma URL, ver imagen de grupo.

Revisamos porque no apunta a la misma URL, ver imagen de grupo.
wbuezas commented 2024-02-27 11:19:09 +00:00
Contributor
Copy Link

Corregido.

Commit: 59dc1a4e38

Corregido. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/59dc1a4e3867a6b722f236113141c7e874aed5f7
jsegarra marked this conversation as resolved
wbuezas added 1 commit 2024-02-27 11:18:06 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
59dc1a4e38 Entry summary icon redirection condition
jsegarra requested changes 2024-02-29 06:55:46 +00:00
src/pages/Entry/Card/EntrySummary.vue Outdated
@ -150,2 +150,3 @@
<template #header-left>
<a class="header link" :href="entryUrl">
<router-link
v-if="route.name !== 'EntrySummary'"
jsegarra commented 2024-02-29 06:55:43 +00:00
Member
Copy Link

Creo que faltaría actualizar el código porque si fusionamos primero entries, supplier dará conflictos, y viceversa

Creo que faltaría actualizar el código porque si fusionamos primero entries, supplier dará conflictos, y viceversa
wbuezas commented 2024-02-29 12:08:28 +00:00
Contributor
Copy Link

Si @jsegarra, coincido totalmente, primero se deberia mergear suppliers y luego actualizar esta rama con esos cambios

Si @jsegarra, coincido totalmente, primero se deberia mergear `suppliers` y luego actualizar esta rama con esos cambios
jsegarra reviewed 2024-03-06 09:39:09 +00:00
src/pages/Entry/Card/EntryBasicData.vue
@ -61,6 +61,7 @@ const onFilterTravelSelected = (formData, id) => {
:url-update="`Entries/${route.params.id}`"
model="entry"
auto-load
:clear-store-on-unmount="false"
jsegarra commented 2024-03-06 09:39:09 +00:00
Member
Copy Link

Ya que tenemos que resolver conflictos, podemos poner el icono relleno con "font-variation-settings: 'FILL' 1;"

Ya que tenemos que resolver conflictos, podemos poner el icono relleno con "font-variation-settings: 'FILL' 1;"
wbuezas commented 2024-03-06 13:00:44 +00:00
Contributor
Copy Link

Añadido!

Commit: fe8e4a8342

Añadido! Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/fe8e4a834274ab1b85ba105c65605dddfdf3a8e8
jsegarra marked this conversation as resolved
wbuezas added 1 commit 2024-03-06 12:53:02 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
fe8e4a8342 add VnSelectDialog refill to action icon
jsegarra requested review from jgallego 2024-03-12 05:48:53 +00:00
jsegarra approved these changes 2024-03-12 05:49:29 +00:00
Dismissed
jsegarra approved these changes 2024-03-12 05:50:40 +00:00
Dismissed
jsegarra approved these changes 2024-03-12 05:50:49 +00:00
Dismissed
jgallego approved these changes 2024-03-12 07:03:10 +00:00
Dismissed
alexm approved these changes 2024-03-12 09:06:48 +00:00
Dismissed
wbuezas added 1 commit 2024-03-13 15:57:51 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
f26d14e875 resolve conflicts
wbuezas dismissed jsegarra’s review 2024-03-13 15:57:51 +00:00
Reason:

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

wbuezas dismissed jsegarra’s review 2024-03-13 15:57:51 +00:00
Reason:

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

wbuezas dismissed jsegarra’s review 2024-03-13 15:57:51 +00:00
Reason:

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

wbuezas dismissed jgallego’s review 2024-03-13 15:57:51 +00:00
Reason:

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

wbuezas dismissed alexm’s review 2024-03-13 15:57:51 +00:00
Reason:

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

alexm approved these changes 2024-03-14 06:37:21 +00:00
alexm merged commit ab8e1fdc04 into dev 2024-03-14 06:39:45 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
jsegarra
jgallego
alexm
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
5 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#201
No description provided.
Delete Branch ":CambiosSolicitadosEntries"

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?