#6942 improve invoiceIn #220

Merged
jorgep merged 63 commits from 6942-improveInvoceIn into dev 2024-05-29 07:03:46 +00:00
Member
No description provided.
jorgep added 1 commit 2024-03-04 07:30:38 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
e10e06bcd4
fix: refs #6942 selectable expense
jorgep added 1 commit 2024-03-04 12:42:03 +00:00
jorgep added 1 commit 2024-03-05 08:04:03 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
0fd83cf435
fix: refs #6942 changes and new features
jorgep added 1 commit 2024-03-05 08:28:10 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
87d458e196
feat: refs #6942 vat rate total
jorgep added 1 commit 2024-03-05 15:31:54 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
5551781bac
feat: refs #6942 to unbook
jorgep added 2 commits 2024-03-06 09:24:11 +00:00
jorgep added 1 commit 2024-03-06 09:27:30 +00:00
jorgep added 1 commit 2024-03-06 09:29:17 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
ec3b44654e
fix: refs #6942 vnLocation
jorgep added 1 commit 2024-03-06 11:36:34 +00:00
jorgep added 1 commit 2024-03-06 13:41:40 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
2e19194a08
fix: refs #6942 reloading
jorgep added 1 commit 2024-03-06 15:28:17 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
9cbffff627
fear: refs #6942 save and continue btn
jorgep added 1 commit 2024-03-06 15:29:14 +00:00
jorgep added 3 commits 2024-03-07 13:59:22 +00:00
jorgep reviewed 2024-03-07 14:08:43 +00:00
@ -27,3 +27,3 @@
type: String,
type: [String, Number],
default: '',
},
Author
Member

Elimino el warning de consola.

Elimino el warning de consola.
jorgep reviewed 2024-03-07 14:08:58 +00:00
@ -154,3 +154,3 @@
:order-by="orderBy"
:sort-by="sortBy"
:fields="fields"
/>
Author
Member

Elimino warning de consola.

Elimino warning de consola.
jorgep added 1 commit 2024-03-07 14:51:41 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
6ed6e4150a
feat: refs #6942 test e2e tobook & toUnbook
jorgep added 1 commit 2024-03-07 14:54:04 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
24700b5441
fix: refs #6942 drop comments
jorgep added 1 commit 2024-03-08 15:58:00 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
db632f3c11
feat: refs #6942 to book summary button & reactive value
jorgep added 1 commit 2024-03-11 07:33:09 +00:00
jorgep changed title from WIP: #6942 improve invoiceIn to #6942 improve invoiceIn 2024-03-11 07:33:34 +00:00
jorgep added the
CR / Tests passed
label 2024-03-11 07:38:37 +00:00
jorgep requested review from jgallego 2024-03-11 07:38:48 +00:00
jorgep requested review from jsegarra 2024-03-11 07:38:48 +00:00
Author
Member

@jgallego Revisar diseño del apartado summary y botón descontabilizar.

@jgallego Revisar diseño del apartado summary y botón descontabilizar.
jorgep added 2 commits 2024-03-11 12:54:30 +00:00
jsegarra approved these changes 2024-03-12 05:43:51 +00:00
Dismissed
jsegarra added spent time 2024-03-12 05:43:58 +00:00
15 hours
jsegarra deleted spent time 2024-03-12 05:44:51 +00:00
- 15 hours
jsegarra added spent time 2024-03-12 05:44:57 +00:00
15 minutes
jgallego requested changes 2024-03-12 06:57:02 +00:00
Dismissed
@ -755,3 +756,4 @@
create: 'Crear',
log: 'Historial',
},
list: {
Owner

Sage transación
Sage transacción

~~Sage transación~~ Sage transacción
jorgep marked this conversation as resolved
@ -429,3 +324,1 @@
</div>
<div class="col"></div>
</div>
</span>
Owner

Cuidado porque aquí si intento descontabilizar desde el descriptor me dice que no puedo pero si lo hago desmarcando el checkbox sí.
Quita visualmente el check y que no se pueda modificar directamente
http://localhost:9000/#/invoice-in/2/basic-data

Cuidado porque aquí si intento descontabilizar desde el descriptor me dice que no puedo pero si lo hago desmarcando el checkbox sí. Quita visualmente el check y que no se pueda modificar directamente http://localhost:9000/#/invoice-in/2/basic-data
Member

Vaya, que mal, había probado el registro 1 y 4 y puedo contabilizar y descontabiliza, en bucle sin fin.
Pero si uso tu registro 2, me sale "El asiento fue enlazado a Sage, por favor contacta con administración"

Vaya, que mal, había probado el registro 1 y 4 y puedo contabilizar y descontabiliza, en bucle sin fin. Pero si uso tu registro 2, me sale "El asiento fue enlazado a Sage, por favor contacta con administración"
Author
Member

Vaya, que mal, había probado el registro 1 y 4 y puedo contabilizar y descontabiliza, en bucle sin fin.
Pero si uso tu registro 2, me sale "El asiento fue enlazado a Sage, por favor contacta con administración"

Es correcto.

> Vaya, que mal, había probado el registro 1 y 4 y puedo contabilizar y descontabiliza, en bucle sin fin. > Pero si uso tu registro 2, me sale "El asiento fue enlazado a Sage, por favor contacta con administración" Es correcto.
Author
Member

Cuidado porque aquí si intento descontabilizar desde el descriptor me dice que no puedo pero si lo hago desmarcando el checkbox sí.
Quita visualmente el check y que no se pueda modificar directamente
http://localhost:9000/#/invoice-in/2/basic-data

hecho. ¿Habría que modificar algún permiso o con quitarlo de basic-data es suficiente? @jgallego

> Cuidado porque aquí si intento descontabilizar desde el descriptor me dice que no puedo pero si lo hago desmarcando el checkbox sí. > Quita visualmente el check y que no se pueda modificar directamente > http://localhost:9000/#/invoice-in/2/basic-data hecho. ¿Habría que modificar algún permiso o con quitarlo de basic-data es suficiente? @jgallego
Owner

quitarlo no es suficiente, hay que quitar permisos, ese campo solo debe poder ser modificado via back

quitarlo no es suficiente, hay que quitar permisos, ese campo solo debe poder ser modificado via back
@ -406,4 +388,4 @@
</template>
<template #body="{ entity }">
<VnLv :label="t('invoiceIn.card.issued')" :value="toDate(entity.issued)" />
<VnLv :label="t('invoiceIn.summary.booked')" :value="toDate(entity.booked)" />
Owner

Este campo en ningún caso me ha sacado valor

Este campo en ningún caso me ha sacado valor
Author
Member

Esto pasa en salix también en local. Creo que lo comente con Carlos Andrés, es por las fixtures.

Esto pasa en salix también en local. Creo que lo comente con Carlos Andrés, es por las fixtures.
Owner

si voy a la seccion http://localhost:9000/#/invoice-in/1/basic-data y cambio el valor de Fecha contable, no actualiza el descriptor, si hago F5 ya lo muestra, deberia hacerlo al apretar guardar en basic-data

si voy a la seccion http://localhost:9000/#/invoice-in/1/basic-data y cambio el valor de Fecha contable, no actualiza el descriptor, si hago F5 ya lo muestra, deberia hacerlo al apretar guardar en basic-data
@ -288,3 +313,2 @@
<QIcon name="open_in_new" />
</a>
<span class="header header-link">{{ t('Totals') }}</span>
</QCardSection>
Owner

lo pongo aqui pero no es esta linea
en la seccion iva del summary en los campos sage vat y sage transaccion quieren en la columna sage vat que aparezca:
8 : H.P. IVA 21% CEE
20 : Adquisic.intracomunitarias de bienes y serv.corr.

es decir, añadir la clave primaria

lo pongo aqui pero no es esta linea en la seccion iva del summary en los campos sage vat y sage transaccion quieren en la columna sage vat que aparezca: 8 : H.P. IVA 21% CEE 20 : Adquisic.intracomunitarias de bienes y serv.corr. es decir, añadir la clave primaria
jorgep marked this conversation as resolved
@ -300,2 +304,4 @@
</QTd>
</template>
<template #bottom-row>
<QTr class="bg">
Owner

En el summary el pie tiene un color de fondo que es un gris mas claro y ayuda a ver los totales. Copialo

En el summary el pie tiene un color de fondo que es un gris mas claro y ayuda a ver los totales. Copialo
Author
Member

@jsegarra @jgallego esta clase la pondría en VnTable cuando se cree.

@jsegarra @jgallego esta clase la pondría en VnTable cuando se cree.
jorgep marked this conversation as resolved
@ -302,0 +315,4 @@
<QTd> {{ toCurrency(getTotalRate(rows)) }}</QTd>
<QTd />
</QTr>
</template>
Owner

En esta seccion en los desplegables Sage iva, transactionTypeSageFk no quieren ver la descripcion solo la clave primaria de la tabla. si despliegan veran el valor

En esta seccion en los desplegables Sage iva, transactionTypeSageFk no quieren ver la descripcion solo la clave primaria de la tabla. si despliegan veran el valor
jorgep marked this conversation as resolved
jorgep added 1 commit 2024-03-12 09:27:49 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
252d9e3a50
fix: refs #6942 table bottom highlight & drop isBooked field
jorgep dismissed jsegarra’s review 2024-03-12 09:27:49 +00:00
Reason:

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

jorgep requested review from jgallego 2024-03-12 09:29:07 +00:00
jgallego added 1 commit 2024-03-13 06:51:14 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
c0a7574cbf
feat: 6942 hashtag in key : value summary
jgallego requested changes 2024-03-13 06:54:09 +00:00
Dismissed
@ -185,6 +187,17 @@ async function insert() {
/>
</QTd>
</template>
<template #bottom-row>
Owner

http://localhost:9000/#/invoice-in/4/due-day al entrar en esta seccion me dice Access Denied en ingles con developer, cuando en realidad esta seccion la deben de poder usar el rol administrative.

http://localhost:9000/#/invoice-in/4/due-day al entrar en esta seccion me dice Access Denied en ingles con developer, cuando en realidad esta seccion la deben de poder usar el rol administrative.
Author
Member

A mi no me da ningún fallo . Compruebalo de nuevo y me dices.

A mi no me da ningún fallo . Compruebalo de nuevo y me dices.
Member

No me da fallo.
Pregunta: la fecha y el placeholder está como yyyy/mm/dd, es correcto?

No me da fallo. Pregunta: la fecha y el placeholder está como yyyy/mm/dd, es correcto?
Author
Member

@jsegarra Ahora se usa VnInputDate.

@jsegarra Ahora se usa VnInputDate.
jorgep marked this conversation as resolved
jorgep added 2 commits 2024-03-15 08:40:01 +00:00
jorgep added 1 commit 2024-03-25 14:03:40 +00:00
jorgep added 1 commit 2024-05-17 12:36:25 +00:00
jorgep added 2 commits 2024-05-17 15:27:01 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
81998a4137
refactor: refs #6942 summary
Member

@jorgep A mi me da error al hacer gulp docker o myt run
image

Después de resolver esto, vuelvo a probar
image

Han deprecado el campo hace 4 días, hablaré con Carlos Andrés.

@jorgep A mi me da error al hacer gulp docker o myt run ![image](/attachments/2fb524a9-92c8-4b16-a7a3-ebfd29b923d4) Después de resolver esto, vuelvo a probar ![image](/attachments/7e313038-4d02-4957-a83e-fd4f92bd40ad) Han deprecado el campo hace 4 días, hablaré con Carlos Andrés.
jsegarra requested changes 2024-05-17 16:18:59 +00:00
Dismissed
@ -1,5 +1,5 @@
<script setup>
import { onBeforeMount, computed, watchEffect } from 'vue';
import { onBeforeMount, computed, watchEffect, watch } from 'vue';
Member

Warning

Warning
jorgep marked this conversation as resolved
@ -319,0 +323,4 @@
{{ toCurrency(entity.totals.totalDueDay) }}
</QChip>
</template>
</VnLv>
Member

Warning en la 338, 366 y 394

Warning en la 338, 366 y 394
jorgep marked this conversation as resolved
Member

Si reemplazo poniendo otro valor me concatena el valor
Reemplazo la primera fila por 22
image
Añado ,2
image

¿Dónde?

Si reemplazo poniendo otro valor me concatena el valor Reemplazo la primera fila por 22 ![image](/attachments/53544624-b0cd-4254-ba69-77264e985be2) Añado ,2 ![image](/attachments/3aa74604-b5fc-49b2-80cf-4b2ce53d431e) ¿Dónde?
4.0 KiB
3.5 KiB
jsegarra requested changes 2024-05-17 16:24:46 +00:00
Dismissed
@ -73,6 +74,7 @@ async function insert() {
await axios.post('/InvoiceInDueDays/new', { id: +invoiceId });
await invoiceInFormRef.value.reload();
}
const getTotalAmount = (rows) => rows.reduce((acc, cur) => acc + cur.amount, 0);
Member

Esta es mi propuesta para el error

const getTotalAmount = (rows) => rows.reduce((acc, { amount }) => acc + +amount, 0);

Esta es mi propuesta para el error const getTotalAmount = (rows) => rows.reduce((acc, { amount }) => acc + +amount, 0);
jorgep marked this conversation as resolved
@ -184,6 +186,17 @@ async function insert() {
/>
</QTd>
Member

Divisa no es seleccionable? Quiero decir estña el campo pero sin acción

Divisa no es seleccionable? Quiero decir estña el campo pero sin acción
Author
Member

Si la divisa es euros no se puede editar ese campo

Si la divisa es euros no se puede editar ese campo
jorgep marked this conversation as resolved
jsegarra requested changes 2024-05-17 16:26:41 +00:00
Dismissed
@ -294,7 +307,11 @@ async function insert() {
<QBtn color="primary" icon="add" size="lg" round @click="insert" />
Member

Que hace la función de insert?
Restaurar la tabla? Eso ya tiene un botón

Que hace la función de insert? Restaurar la tabla? Eso ya tiene un botón
Author
Member

Llama al back InvoiceInDueDays/new (llama a invoiceIn_calculate) y recarga el formulario...

Llama al back InvoiceInDueDays/new (llama a invoiceIn_calculate) y recarga el formulario...
jorgep marked this conversation as resolved
jorgep added 1 commit 2024-05-20 06:55:43 +00:00
Author
Member

@jsegarra de momento yo no corregiría esta rama, aviso cuando este para corregir.

@jsegarra de momento yo no corregiría esta rama, aviso cuando este para corregir.
jorgep added 1 commit 2024-05-20 07:31:34 +00:00
jorgep added 1 commit 2024-05-20 07:32:06 +00:00
jorgep changed title from #6942 improve invoiceIn to WIP: #6942 improve invoiceIn 2024-05-20 08:09:19 +00:00
jorgep added 1 commit 2024-05-21 07:05:10 +00:00
jorgep added 1 commit 2024-05-22 07:17:25 +00:00
jorgep added 1 commit 2024-05-22 09:22:50 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
5bddc6e04d
fix: refs #6942 wip: formModel
jorgep added 2 commits 2024-05-22 14:46:12 +00:00
jorgep added 1 commit 2024-05-23 07:01:01 +00:00
jorgep added 3 commits 2024-05-23 10:33:15 +00:00
jorgep added 1 commit 2024-05-23 12:04:39 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
4a6d09ae2c
feat: refs #6942 use correct currency in InvoiceIn components
jorgep added 1 commit 2024-05-23 12:19:42 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
d153cc9c24
fix: refs #6942 unit tests
jorgep reviewed 2024-05-23 14:19:28 +00:00
@ -60,6 +62,11 @@ const $props = defineProps({
type: Function,
default: null,
},
goTo: {
Author
Member

url a la que redirigir al hacer click en 'save and continue'

url a la que redirigir al hacer click en 'save and continue'
jorgep reviewed 2024-05-23 14:20:18 +00:00
@ -310,7 +322,40 @@ watch(formUrl, async () => {
:title="t('globals.reset')"
v-if="$props.defaultReset"
/>
<QBtnDropdown
Author
Member

Botón desplegable con la op. de guardar y continuar o solo guardar.

Botón desplegable con la op. de guardar y continuar o solo guardar.
jorgep reviewed 2024-05-23 14:20:57 +00:00
@ -74,55 +76,17 @@ const $props = defineProps({
type: Function,
default: null,
},
goTo: {
Author
Member

Lo mismo que en crudModel

Lo mismo que en crudModel
jorgep reviewed 2024-05-23 14:21:40 +00:00
@ -145,2 +109,3 @@
}));
const startFormWatcher = () => {
onMounted(async () => {
Author
Member

Lo he pasado aquí para que todos los hooks estén agrupados.

Lo he pasado aquí para que todos los hooks estén agrupados.
jorgep reviewed 2024-05-23 14:22:29 +00:00
@ -146,1 +110,3 @@
const startFormWatcher = () => {
onMounted(async () => {
originalData.value = JSON.parse(JSON.stringify($props.formInitialData ?? {}));
Author
Member

Hay que hacerlo así para que se haga una copia, si no, se hace una referencia al mismo objeto.

Hay que hacerlo así para que se haga una copia, si no, se hace una referencia al mismo objeto.
Member

Se podría poner el comentario en código?

Se podría poner el comentario en código?
jorgep reviewed 2024-05-23 14:24:21 +00:00
@ -147,0 +126,4 @@
if (!oldVal) return;
hasChanges.value =
!isResetting.value &&
JSON.stringify(newVal) !== JSON.stringify(originalData.value);
Author
Member

Para ver si hay cambios comparamos el obj actual con el original. Con la lógica anterior, no funcionaba bien si se usa una store, ya que al emitir este valor se 'actualiza'. Así se hace la comprobación correcta.

Para ver si hay cambios comparamos el obj actual con el original. Con la lógica anterior, no funcionaba bien si se usa una store, ya que al emitir este valor se 'actualiza'. Así se hace la comprobación correcta.
Member

Se podría poner el comentario en código?

Se podría poner el comentario en código?
Author
Member

yo lo veo bien sin comentarios. lo que diga @jgallego

yo lo veo bien sin comentarios. lo que diga @jgallego
Owner

Yo no pondría comentarios, pero si en otros sitios está implementado distinto yo lo cambiaría para que quien venga detrás coja siempre una buena implementación

Yo no pondría comentarios, pero si en otros sitios está implementado distinto yo lo cambiaría para que quien venga detrás coja siempre una buena implementación
jorgep reviewed 2024-05-23 14:25:02 +00:00
@ -154,2 +141,3 @@
);
};
watch(formUrl, async () => {
Author
Member

Agrupado con el resto de hooks.

Agrupado con el resto de hooks.
jorgep reviewed 2024-05-23 14:25:17 +00:00
@ -156,0 +146,4 @@
await fetch();
});
onBeforeRouteLeave((to, from, next) => {
Author
Member

Agrupado con el resto de hooks

Agrupado con el resto de hooks
jorgep reviewed 2024-05-23 14:26:03 +00:00
@ -182,2 +187,4 @@
try {
const body = $props.mapper ? $props.mapper(formData.value) : formData.value;
const method = $props.urlCreate ? 'post' : 'patch';
const url =
Author
Member

En mi opinión queda mucho más legible así, aunque solo se use 1 vez.

En mi opinión queda mucho más legible así, aunque solo se use 1 vez.
jorgep reviewed 2024-05-23 14:28:22 +00:00
@ -231,3 +237,1 @@
reset();
fetch();
});
function updateAndEmit(val, evt) {
Author
Member

Esta parte se repetía varias veces

Esta parte se repetía varias veces
jorgep reviewed 2024-05-23 14:29:05 +00:00
@ -273,10 +279,42 @@ defineExpose({
:disable="!hasChanges"
:title="t(defaultButtons.reset.label)"
/>
<QBtnDropdown
Author
Member

Lo mismo que crudModel

Lo mismo que crudModel
jorgep reviewed 2024-05-23 14:29:51 +00:00
@ -43,3 +43,3 @@
onBeforeRouteUpdate(async (to, from) => {
if (to.params.id !== from.params.id) {
arrayData.store.url = `${props.baseUrl}/${route.params.id}`;
arrayData.store.url = `${props.baseUrl}/${to.params.id}`;
Author
Member

Al no haber cambiado de ruta, se estaba poniendo el mismo id actual.

Al no haber cambiado de ruta, se estaba poniendo el mismo id actual.
jorgep reviewed 2024-05-23 14:33:32 +00:00
@ -58,0 +57,4 @@
store = arrayData.store;
entity = computed(() => (Array.isArray(store.data) ? store.data[0] : store.data));
if ($props.dataKey !== useRoute().meta.moduleName) await getData();
Author
Member

Permite cargar los datos solo una vez si el módulo es el mismo que dataKey

Permite cargar los datos solo una vez si el módulo es el mismo que dataKey
Member

Se podría poner el comentario en código?

Se podría poner el comentario en código?
jorgep reviewed 2024-05-23 14:33:51 +00:00
@ -58,0 +59,4 @@
if ($props.dataKey !== useRoute().meta.moduleName) await getData();
watch(
() => [$props.url, $props.filter],
Author
Member

Solo interesa ver cambios en estas 2 props.

Solo interesa ver cambios en estas 2 props.
jorgep reviewed 2024-05-23 14:35:29 +00:00
@ -133,3 +145,3 @@
<div class="title">
<span v-if="$props.title" :title="$props.title">
{{ $props.title }}
{{ entity[title] ?? $props.title }}
Author
Member

Ahora, cuando el descriptor coja los datos directamente desde vncard, se le puede indicar que propiedad usar. Se mantiene la compatibilidad con las secciones que todavía no la usan, pudiendoles pasarle directamente el valor.

Ahora, cuando el descriptor coja los datos directamente desde vncard, se le puede indicar que propiedad usar. Se mantiene la compatibilidad con las secciones que todavía no la usan, pudiendoles pasarle directamente el valor.
jorgep reviewed 2024-05-23 14:41:21 +00:00
@ -85,2 +72,2 @@
);
}
const getTotal = (data, key) =>
data.reduce((acc, cur) => acc + +String(cur[key]).replace(',', '.'), 0);
Author
Member

Si se escribe la coma, te devuelve NaN

Si se escribe la coma, te devuelve NaN
jorgep reviewed 2024-05-23 14:41:58 +00:00
@ -123,3 +92,2 @@
url="InvoiceInIntrastats"
auto-load
:data-required="{ invoiceInFk: route.params.id }"
:auto-load="!currency"
Author
Member

espera a que se carguen los datos de arrayData, ya que hace falta saber el currency

espera a que se carguen los datos de arrayData, ya que hace falta saber el currency
jorgep reviewed 2024-05-23 14:43:43 +00:00
@ -4,33 +4,17 @@ import { useI18n } from 'vue-i18n';
import VnSelect from 'components/common/VnSelect.vue';
Author
Member

Se han reordenado los campos en base a como están en Salix.

Se han reordenado los campos en base a como están en Salix.
jorgep reviewed 2024-05-23 14:44:13 +00:00
@ -94,1 +52,4 @@
</QItem>
<QItem>
<QItemSection>
<VnSelect
Author
Member

Se corrige el filtrado de suppliers.

Se corrige el filtrado de suppliers.
jorgep reviewed 2024-05-23 14:45:56 +00:00
@ -19,3 +19,3 @@
cy.visit(`/#/invoice-in/1/vat`);
cy.getValue(firstLineVat).should('equal', 'H.P. IVA 21% CEE');
cy.getValue(firstLineVat).should('equal', '8');
Author
Member

Se está comprobando el valor del input, no el "label"

Se está comprobando el valor del input, no el "label"
jorgep reviewed 2024-05-23 14:46:40 +00:00
@ -92,8 +92,13 @@ Cypress.Commands.add('checkOption', (selector) => {
// Global buttons
Cypress.Commands.add('saveCard', () => {
Author
Member

Se adapta para que te deje guardar en caso de haber un desplegable

Se adapta para que te deje guardar en caso de haber un desplegable
jorgep reviewed 2024-05-23 14:47:03 +00:00
@ -1,34 +0,0 @@
import { vi, describe, expect, it, beforeAll } from 'vitest';
Author
Member

Ya se comprueba en e2e

Ya se comprueba en e2e
jorgep reviewed 2024-05-23 14:47:48 +00:00
@ -20,3 +20,3 @@
describe('getTotal()', () => {
it('should correctly handle the sum', () => {
vm.invoceInIntrastat = [
const invoceInIntrastat = [
Author
Member

Ahora a la fn hay que pasarle el array. Así los valoraes se actualizan de forma reactiva.

Ahora a la fn hay que pasarle el array. Así los valoraes se actualizan de forma reactiva.
jorgep added 1 commit 2024-05-23 14:48:53 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
b6638e3545
refactor: refs #6942 use router hook
jorgep added 1 commit 2024-05-23 14:49:26 +00:00
jorgep added 1 commit 2024-05-24 07:02:44 +00:00
jorgep added 1 commit 2024-05-24 07:56:20 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
1c76d86710
fix: refs #6942 e2e tests
jorgep added 1 commit 2024-05-24 09:43:58 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
d460b198a4
fix: refs #6942 skip e2e tests
jorgep added 1 commit 2024-05-24 10:10:31 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
933e0be028
fix: refs #6942 e2e tests
jorgep changed title from WIP: #6942 improve invoiceIn to #6942 improve invoiceIn 2024-05-24 10:27:18 +00:00
jorgep requested review from jgallego 2024-05-24 10:29:13 +00:00
jorgep requested review from jsegarra 2024-05-24 10:29:17 +00:00
jgallego requested changes 2024-05-24 15:06:01 +00:00
Dismissed
@ -199,6 +199,7 @@ function getLogTree(data) {
nLogs >= 6;
if (modelChanged) {
console.log(locale);
Owner

?

?
Author
Member

Se me ha colado. Lo quito.

Se me ha colado. Lo quito.
jorgep marked this conversation as resolved
@ -184,3 +184,3 @@
</QBtn>
<QBtn
:href="`${url}invoice-in/create?supplierFk=${entity.id}`"
:href="`#/invoice-in/create?supplierFk=${entity.id}`"
Owner

Esta en la ruta actual?

Esta en la ruta actual?
Author
Member

Sí, antes redirigia a Salix.

Sí, antes redirigia a Salix.
jorgep marked this conversation as resolved
jorgep added 2 commits 2024-05-27 06:57:50 +00:00
jorgep added 1 commit 2024-05-27 07:07:26 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
b6f11096f3
feat: refs #6942 url update
jorgep requested review from jgallego 2024-05-27 08:49:36 +00:00
jsegarra started working 2024-05-27 09:58:01 +00:00
Member

Hay 6 warnings, en la medida de lo posible y tiempo, resolver

Hay 6 warnings, en la medida de lo posible y tiempo, resolver
jsegarra requested changes 2024-05-27 10:49:13 +00:00
Dismissed
@ -19,3 +20,2 @@
const editDownloadDisabled = ref(false);
const arrayData = useArrayData('InvoiceIn');
const invoiceIn = computed(() => arrayData.store.data);
const invoiceIn = computed(() => useArrayData('InvoiceIn').store.data);
Member

Usar useRoute().meta.moduleName

Usar useRoute().meta.moduleName
jorgep marked this conversation as resolved
@ -33,3 +28,3 @@
const { t } = useI18n();
const { openReport, sendEmail } = usePrintService();
const arrayData = useArrayData('InvoiceIn');
const { store } = useArrayData('InvoiceIn');
Member

useRoute().meta.moduleName

useRoute().meta.moduleName
jorgep marked this conversation as resolved
@ -45,10 +40,9 @@ const cplusRectificationTypes = ref([]);
const siiTypeInvoiceOuts = ref([]);
const invoiceCorrectionTypes = ref([]);
const actions = {
Member

Hay 3 acciones y el texto es el mismo, menos el verbo de la acción.

Se puede hacer una traducción con parámetro

Hay 3 acciones y el texto es el mismo, menos el verbo de la acción. Se puede hacer una traducción con parámetro
jorgep marked this conversation as resolved
Author
Member

Hay 6 warnings, en la medida de lo posible y tiempo, resolver

Ninguno es relacionado con mi tarea.

> Hay 6 warnings, en la medida de lo posible y tiempo, resolver Ninguno es relacionado con mi tarea.
jgallego approved these changes 2024-05-27 11:11:05 +00:00
Dismissed
jorgep added 1 commit 2024-05-27 11:36:19 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
ef9f8b8705
refactor: refs #6942 store key & actions
jorgep dismissed jgallego’s review 2024-05-27 11:36:19 +00:00
Reason:

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

jorgep added 1 commit 2024-05-27 11:36:40 +00:00
jorgep requested review from jsegarra 2024-05-27 11:36:59 +00:00
jorgep added 1 commit 2024-05-27 12:37:03 +00:00
jorgep added 1 commit 2024-05-27 12:51:26 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
90cebb85fe
reafactor: refs #6942 use meta moduleName
jorgep added 1 commit 2024-05-27 12:53:53 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
df6e6c66a9
fix: refs #6942 import
jsegarra stopped working 2024-05-27 13:02:10 +00:00
3 hours 4 minutes
jorgep added 2 commits 2024-05-28 07:01:38 +00:00
jorgep added 1 commit 2024-05-28 09:59:18 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
5aa3b14014
refactor: refs #6942 WIP summary layout
jorgep added 1 commit 2024-05-28 11:11:42 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
3b3e7e7e03
fix: refs #6942 tests & summary table spacing
jorgep added 1 commit 2024-05-28 11:14:18 +00:00
jorgep reviewed 2024-05-28 11:16:27 +00:00
@ -16,41 +16,6 @@ describe('InvoiceInVat', () => {
}).vm;
});
describe('addExpense()', () => {
Author
Member

ya se comprueba en e2e

ya se comprueba en e2e
jorgep reviewed 2024-05-28 11:17:15 +00:00
@ -24,6 +24,7 @@ vi.mock('vue-router', () => ({
params: {
id: 1,
},
meta: { moduleName: 'mockName' },
Author
Member

Lo pongo en ambos sitios ya de paso.

Lo pongo en ambos sitios ya de paso.
jsegarra approved these changes 2024-05-29 06:49:03 +00:00
jorgep merged commit f15e8e42b3 into dev 2024-05-29 07:03:46 +00:00
jorgep deleted branch 6942-improveInvoceIn 2024-05-29 07:03:46 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
3 Participants
Notifications
Total Time Spent: 3 hours 19 minutes
jsegarra
3 hours 19 minutes
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#220
No description provided.