fix: refs #6900 fine tunning #661

Merged
jorgep merged 13 commits from 6900-hotfix-fineTunning into master 2024-09-04 13:58:59 +00:00
Member
No description provided.
jorgep added 1 commit 2024-09-02 06:46:05 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
be63c8a7fa
fix: refs #6900 fine tunning wip
jorgep added 1 commit 2024-09-02 11:15:11 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
f24daa4ebd
feat: refs #6900 hide menu item
jorgep added 1 commit 2024-09-02 11:32:25 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
8ea9742068
chore: refs #6900 rollback
jorgep added 1 commit 2024-09-02 11:39:17 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
136f9abeff
chore: refs #6900 rollback
jorgep reviewed 2024-09-02 11:41:32 +00:00
@ -29,6 +29,15 @@ onMounted(async () => {
getRoutes();
});
watch(
Author
Member

Si se realiza algún cambio en el router, se vuelven a crear los items. Esto lo hago para que si enuna ruta se modifica meta.hidden , se oculte o se muestre.

Si se realiza algún cambio en el router, se vuelven a crear los items. Esto lo hago para que si enuna ruta se modifica **meta.hidden** , se oculte o se muestre.
jorgep reviewed 2024-09-02 11:41:54 +00:00
@ -0,0 +1,10 @@
import { toCurrency } from 'src/filters';
Author
Member

Lo uso varias veces.

Lo uso varias veces.
jorgep reviewed 2024-09-02 11:45:06 +00:00
@ -223,6 +223,10 @@ async function onSubmit() {
autofocus
/>
</VnRow>
<VnRow>
Author
Member

Ahora están todas las fechas juntas.

Ahora están todas las fechas juntas.
jorgep reviewed 2024-09-02 11:46:06 +00:00
@ -21,2 +23,4 @@
],
};
onBeforeRouteUpdate(async (to) => {
Author
Member

En caso de cambiar de factura, se consulta si es rectificativa, si no lo es, se oculta del menu lateral.

En caso de cambiar de factura, se consulta si es rectificativa, si no lo es, se oculta del menu lateral.
jorgep reviewed 2024-09-02 11:46:33 +00:00
@ -360,3 +359,1 @@
:label="t('invoiceIn.card.amount')"
:value="toCurrency(totalAmount, entity.currency?.code)"
/>
<VnLv :label="t('invoiceIn.card.amount')" :value="toCurrency(totalAmount)" />
Author
Member

Este valor siempre en euros.

Este valor siempre en euros.
jorgep reviewed 2024-09-02 11:48:16 +00:00
@ -36,3 +36,3 @@
label: 'invoiceIn.summary.taxableBase',
field: (row) => row.taxableBase,
format: (value) => toCurrency(value, currency.value),
format: (value) => toCurrency(value),
Author
Member

valor en euros.

valor en euros.
jorgep reviewed 2024-09-02 11:48:32 +00:00
@ -73,3 +73,3 @@
label: 'invoiceIn.summary.currency',
field: (row) => row.foreignValue,
format: (value) => value,
format: (val) => val && toCurrency(val, currency.value),
Author
Member

valor en otra divisa.

valor en otra divisa.
jorgep reviewed 2024-09-02 11:48:53 +00:00
@ -298,3 +292,1 @@
<VnLv
label="Total"
:value="toCurrency(entity.totals.totalVat, currency)"
:value="toCurrency(entity.totals.totalTaxableBase)"
Author
Member

solo euros.

solo euros.
jorgep reviewed 2024-09-02 11:51:34 +00:00
@ -63,6 +64,22 @@ export default {
path: ':id',
component: () => import('src/pages/InvoiceIn/Card/InvoiceInCard.vue'),
redirect: { name: 'InvoiceInSummary' },
beforeEnter: async (to, from, next) => {
Author
Member

Al entrar se comprueba si es rectificativa. Lamentablemente, la opción de onBeforeRouteLeave, solo está disponible dentro de un componente.

Al entrar se comprueba si es rectificativa. Lamentablemente, la opción de onBeforeRouteLeave, solo está disponible dentro de un componente.
jorgep reviewed 2024-09-02 11:51:48 +00:00
@ -56,6 +56,7 @@ export const useNavigationStore = defineStore('navigationStore', () => {
function addMenuItem(module, route, parent) {
const { meta } = route;
let { menuChildren = null } = meta;
if (meta.hidden) return;
Author
Member

No lo añade.

No lo añade.
jorgep added 1 commit 2024-09-02 11:52:17 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
0faa42867d
chore: refs #6900 drop comment
jorgep changed title from WIP: fix: refs #6900 fine tunning wip to fix: refs #6900 fine tunning wip 2024-09-02 12:10:14 +00:00
jorgep changed title from fix: refs #6900 fine tunning wip to fix: refs #6900 fine tunning 2024-09-02 12:13:18 +00:00
jorgep requested review from jgallego 2024-09-02 14:47:27 +00:00
jorgep added the
CR / Tests passed
label 2024-09-02 14:47:31 +00:00
jorgep added 2 commits 2024-09-02 14:48:20 +00:00
jorgep reviewed 2024-09-02 14:49:07 +00:00
@ -132,40 +123,6 @@ function taxRate(invoiceInTax) {
return (taxTypeSage / 100) * taxableBase;
}
async function addExpense() {
Author
Member

Pasado a componente CreateNewExpenseForm. Este ya tiene la lógica de FormModel para realizar una petición POST.

Pasado a componente **CreateNewExpenseForm**. Este ya tiene la lógica de FormModel para realizar una petición POST.
jorgep reviewed 2024-09-02 14:49:46 +00:00
@ -162,4 +126,0 @@
});
}
}
const getTotalTaxableBase = (rows) =>
Author
Member

Ahora se usa un composable.

Ahora se usa un composable.
jorgep reviewed 2024-09-02 14:50:39 +00:00
@ -442,44 +401,6 @@ const formatOpt = (row, { model, options }, prop) => {
</QTable>
</template>
</CrudModel>
<QDialog ref="newExpenseRef">
Author
Member

Pasado a componente CreateNewExpenseForm

Pasado a componente CreateNewExpenseForm
jorgep reviewed 2024-09-02 14:51:05 +00:00
@ -38,3 +38,2 @@
it('should throw an error creating a new dms if a file is not attached', () => {
cy.get(formInputs).eq(5).click();
cy.get(formInputs).eq(5).type('{selectall}{backspace}');
cy.get(formInputs).eq(7).type('{selectall}{backspace}');
Author
Member

Se ha cambiado el orden de los inputs de basic data.

Se ha cambiado el orden de los inputs de basic data.
jorgep reviewed 2024-09-02 14:52:50 +00:00
@ -11,2 +10,4 @@
cy.login('developer');
cy.visit(`/#/invoice-in/1/vat`);
cy.intercept('GET', '/api/InvoiceIns/1/getTotals').as('lastCall');
cy.wait('@lastCall');
Author
Member

En cypress, si abres un diálogo y modificas la url, este se cierra. He comprobado que en el navegador no se cierre. Si espero a que se hagan todas las llamadas funciona bien, me imagino que es porque cypress va lentorro.

En cypress, si abres un diálogo y modificas la url, este se cierra. He comprobado que en el navegador no se cierre. Si espero a que se hagan todas las llamadas funciona bien, me imagino que es porque cypress va lentorro.
jorgep reviewed 2024-09-02 14:53:44 +00:00
@ -29,4 +30,0 @@
cy.removeRow(2);
});
it('should throw an error if there are fields undefined', () => {
Author
Member

Ahora se usa la prop rules, esta ya válida que se rellenen los campos obligatorios e impide que se envíe en caso contrario.

Ahora se usa la prop **rules**, esta ya válida que se rellenen los campos obligatorios e impide que se envíe en caso contrario.
jgallego requested changes 2024-09-02 17:33:48 +00:00
Dismissed
jgallego left a comment
Owner

mañana hay test-master, si no es necesario mejor súbelo a test y cuando suban todo que suba a master. Si ya estaba en test, no habría que hacer nada.

mañana hay test-master, si no es necesario mejor súbelo a test y cuando suban todo que suba a master. Si ya estaba en test, no habría que hacer nada.
jorgep added 1 commit 2024-09-03 07:07:42 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
02383f28f0
Merge branch 'master' into 6900-hotfix-fineTunning
Author
Member

ya se ha hecho la subida a master, lo subimos a master?

ya se ha hecho la subida a master, lo subimos a master?
jorgep requested review from jgallego 2024-09-03 07:08:30 +00:00
jgallego requested review from jsegarra 2024-09-03 09:31:15 +00:00
jgallego approved these changes 2024-09-03 09:31:37 +00:00
Dismissed
jgallego left a comment
Owner

como va a master pongo tb a segarra por si se me han pasado cosas

como va a master pongo tb a segarra por si se me han pasado cosas
jsegarra requested changes 2024-09-03 10:36:22 +00:00
Dismissed
@ -0,0 +5,4 @@
import FormModelPopup from './FormModelPopup.vue';
const emit = defineEmits(['onDataSaved']);
const { t } = useI18n();
Member

porque no usas $t directamente en el template?

porque no usas $t directamente en el template?
Author
Member

Si son traducciones locales(declaradas dentro del mismo componente), no funciona.

Si son traducciones locales(declaradas dentro del mismo componente), no funciona.
Member

Okey, te lo compro pero las traducciones de code y description están definidas como global
Por tanto, eliminar del componente

Okey, te lo compro pero las traducciones de code y description están definidas como global Por tanto, eliminar del componente
jorgep marked this conversation as resolved
@ -0,0 +2,4 @@
export function getTotal(rows, key, opts = {}) {
const { currency, cb } = opts;
const total = rows.reduce((acc, row) => acc + +(cb ? cb(row) : row[key] || 0), 0);
Member

Porque no sacas el valor cb fuera? y asi simplificas el reduce
Por ciertpo, que significa cb? callback, no vedad?

Porque no sacas el valor cb fuera? y asi simplificas el reduce Por ciertpo, que significa cb? callback, no vedad?
Author
Member

No acabo de entender lo de sacarlo fuera, la idea es que si en lugar de solo sumar, quiero llamar a una fn tipo saca el 20% o 30% lo puedas hacer pasandole un callback(cb)

No acabo de entender lo de sacarlo fuera, la idea es que si en lugar de solo sumar, quiero llamar a una fn tipo saca el 20% o 30% lo puedas hacer pasandole un callback(cb)
Member

Lo dejamos asi

Lo dejamos asi
jsegarra marked this conversation as resolved
@ -22,1 +24,4 @@
};
onBeforeRouteUpdate(async (to) => {
const card = to.matched.find((route) => route.name === 'InvoiceInCard');
Member

Porque :key="$router.path" no sirve?

Porque :key="$router.path" no sirve?
Author
Member

No, no detecta el cambio.

No, no detecta el cambio.
jsegarra marked this conversation as resolved
@ -155,3 +153,3 @@
<QTd />
<QTd>
{{ toCurrency(getTotal(rows, 'amount'), currency) }}
{{ getTotal(rows, 'amount', { currency: 'default' }) }}
Member

En invoiceInDueDay, los parametros son los mismo, declaramos valores por defecto?

En invoiceInDueDay, los parametros son los mismo, declaramos valores por defecto?
Author
Member

No entiendo lo que quieres decir.

No entiendo lo que quieres decir.
Member

El composable getTotal, tiene un 3er parametro vacio.
A lo mejor podriamos reemplzar opts={} por opts = {currency: 'default'}

El composable getTotal, tiene un 3er parametro vacio. A lo mejor podriamos reemplzar opts={} por opts = {currency: 'default'}
Author
Member

No siempre se que quiere devolver el total en valor monetario. A veces solo se quiere hacer la suma de cantidades.

No siempre se que quiere devolver el total en valor monetario. A veces solo se quiere hacer la suma de cantidades.
jorgep added 2 commits 2024-09-03 10:50:54 +00:00
jorgep added 1 commit 2024-09-03 11:03:42 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
6eb5435e25
chore: refs #6900 drop useless code
jorgep dismissed jgallego’s review 2024-09-03 11:03:42 +00:00
Reason:

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

jsegarra requested changes 2024-09-03 11:47:35 +00:00
Dismissed
@ -0,0 +18,4 @@
<template #form-inputs="{ data, validate }">
<VnRow>
<VnInput
:label="`${t('Code')}`"
Member

global.code

global.code
jorgep marked this conversation as resolved
@ -0,0 +33,4 @@
</VnRow>
<VnRow>
<VnInput
:label="`${t('Description')}`"
Member

globals.description

globals.description
jorgep marked this conversation as resolved
jsegarra requested changes 2024-09-03 12:00:46 +00:00
Dismissed
jsegarra left a comment
Member

Me falta tooltip del

Me falta tooltip del
@ -2,18 +2,17 @@
import { ref, computed } from 'vue';
Member

En el boton + de la tabla me falta un tooltip

En el boton + de la tabla me falta un tooltip
jorgep marked this conversation as resolved
@ -226,2 +179,2 @@
class="cursor-pointer"
size="xs"
<template #form>
<CreateNewExpenseForm
Member

Juraría que el botón del mas puede llevar tooltip

Juraría que el botón del mas puede llevar tooltip
jorgep marked this conversation as resolved
jorgep added 1 commit 2024-09-03 12:37:27 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
00beeaf51b
feat: refs #6900 add tooltip
jorgep requested review from jsegarra 2024-09-03 12:37:36 +00:00
jsegarra approved these changes 2024-09-03 14:05:39 +00:00
Dismissed
jsegarra left a comment
Member

Después de la migración, me gastaría reservar un jueves después de almorzar en una sala para analizar esta funcionalidad.
Para un caso concreto OK, pero cuando hayan mas...REFACTOR

Después de la migración, me gastaría reservar un jueves después de almorzar en una sala para analizar esta funcionalidad. Para un caso concreto OK, pero cuando hayan mas...REFACTOR
Author
Member

Después de la migración, me gastaría reservar un jueves después de almorzar en una sala para analizar esta funcionalidad.
Para un caso concreto OK, pero cuando hayan mas...REFACTOR

Vale, pon fecha si quieres y lo comentamos. Yo sigo viendo muy legible y en la línea de la lógica actual(meta.roles, meta.title) usar un parámetro del meta.

> Después de la migración, me gastaría reservar un jueves después de almorzar en una sala para analizar esta funcionalidad. > Para un caso concreto OK, pero cuando hayan mas...REFACTOR Vale, pon fecha si quieres y lo comentamos. Yo sigo viendo muy legible y en la línea de la lógica actual(**meta.roles**, meta.title) usar un parámetro del meta.
Author
Member

Me espero a mañana para fusionar, debido a los problemas que han habido hoy con las transacciones

Me espero a mañana para fusionar, debido a los problemas que han habido hoy con las transacciones
jsegarra requested changes 2024-09-04 06:18:00 +00:00
Dismissed
jsegarra left a comment
Member

He encontrado una alternativa al onBeforeRouter
De todas maneras, tambien lo cambio porque dijiomos de mover la misma logica al composable

He encontrado una alternativa al onBeforeRouter De todas maneras, tambien lo cambio porque dijiomos de mover la misma logica al composable
jorgep added 1 commit 2024-09-04 08:30:19 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
763cf2e345
refactor: refs #6900 create setRectificative composable
jorgep requested review from jsegarra 2024-09-04 08:31:16 +00:00
Member

@jorgep tengo el de @jgallego para fusionar y cuando vuelva @alexm , juntarnos y revisar el mejor enfoque

Te parece bien?
Si es que si, dejo por tu parte, crear una reunión con este tema

@jorgep tengo el ✅ de @jgallego para fusionar y cuando vuelva @alexm , juntarnos y revisar el mejor enfoque Te parece bien? Si es que si, dejo por tu parte, crear una reunión con este tema
Author
Member

@jorgep tengo el de @jgallego para fusionar y cuando vuelva @alexm , juntarnos y revisar el mejor enfoque

Te parece bien?
Si es que si, dejo por tu parte, crear una reunión con este tema

Vale

> @jorgep tengo el ✅ de @jgallego para fusionar y cuando vuelva @alexm , juntarnos y revisar el mejor enfoque > > Te parece bien? > Si es que si, dejo por tu parte, crear una reunión con este tema Vale
jsegarra approved these changes 2024-09-04 13:09:08 +00:00
jorgep merged commit a5d32895ea into master 2024-09-04 13:58:59 +00:00
jorgep deleted branch 6900-hotfix-fineTunning 2024-09-04 13:58:59 +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#661
No description provided.