fix: refs #6900 fine tunning #661
No reviewers
Labels
No Milestone
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix-front#661
Loading…
Reference in New Issue
No description provided.
Delete Branch "6900-hotfix-fineTunning"
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?
@ -29,6 +29,15 @@ onMounted(async () => {
getRoutes();
});
watch(
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.
@ -0,0 +1,10 @@
import { toCurrency } from 'src/filters';
Lo uso varias veces.
@ -223,6 +223,10 @@ async function onSubmit() {
autofocus
/>
</VnRow>
<VnRow>
Ahora están todas las fechas juntas.
@ -21,2 +23,4 @@
],
};
onBeforeRouteUpdate(async (to) => {
En caso de cambiar de factura, se consulta si es rectificativa, si no lo es, se oculta del menu lateral.
@ -360,3 +359,1 @@
:label="t('invoiceIn.card.amount')"
:value="toCurrency(totalAmount, entity.currency?.code)"
/>
<VnLv :label="t('invoiceIn.card.amount')" :value="toCurrency(totalAmount)" />
Este valor siempre en euros.
@ -36,3 +36,3 @@
label: 'invoiceIn.summary.taxableBase',
field: (row) => row.taxableBase,
format: (value) => toCurrency(value, currency.value),
format: (value) => toCurrency(value),
valor en euros.
@ -73,3 +73,3 @@
label: 'invoiceIn.summary.currency',
field: (row) => row.foreignValue,
format: (value) => value,
format: (val) => val && toCurrency(val, currency.value),
valor en otra divisa.
@ -298,3 +292,1 @@
<VnLv
label="Total"
:value="toCurrency(entity.totals.totalVat, currency)"
:value="toCurrency(entity.totals.totalTaxableBase)"
solo euros.
@ -63,6 +64,22 @@ export default {
path: ':id',
component: () => import('src/pages/InvoiceIn/Card/InvoiceInCard.vue'),
redirect: { name: 'InvoiceInSummary' },
beforeEnter: async (to, from, next) => {
Al entrar se comprueba si es rectificativa. Lamentablemente, la opción de onBeforeRouteLeave, solo está disponible dentro de un componente.
@ -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;
No lo añade.
WIP: fix: refs #6900 fine tunning wipto fix: refs #6900 fine tunning wipfix: refs #6900 fine tunning wipto fix: refs #6900 fine tunning@ -132,40 +123,6 @@ function taxRate(invoiceInTax) {
return (taxTypeSage / 100) * taxableBase;
}
async function addExpense() {
Pasado a componente CreateNewExpenseForm. Este ya tiene la lógica de FormModel para realizar una petición POST.
@ -162,4 +126,0 @@
});
}
}
const getTotalTaxableBase = (rows) =>
Ahora se usa un composable.
@ -442,44 +401,6 @@ const formatOpt = (row, { model, options }, prop) => {
</QTable>
</template>
</CrudModel>
<QDialog ref="newExpenseRef">
Pasado a componente CreateNewExpenseForm
@ -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}');
Se ha cambiado el orden de los inputs de basic data.
@ -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');
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.
@ -29,4 +30,0 @@
cy.removeRow(2);
});
it('should throw an error if there are fields undefined', () => {
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.
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.
ya se ha hecho la subida a master, lo subimos a master?
como va a master pongo tb a segarra por si se me han pasado cosas
@ -0,0 +5,4 @@
import FormModelPopup from './FormModelPopup.vue';
const emit = defineEmits(['onDataSaved']);
const { t } = useI18n();
porque no usas $t directamente en el template?
Si son traducciones locales(declaradas dentro del mismo componente), no funciona.
Okey, te lo compro pero las traducciones de code y description están definidas como global
Por tanto, eliminar del componente
@ -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);
Porque no sacas el valor cb fuera? y asi simplificas el reduce
Por ciertpo, que significa cb? callback, no vedad?
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)
Lo dejamos asi
@ -22,1 +24,4 @@
};
onBeforeRouteUpdate(async (to) => {
const card = to.matched.find((route) => route.name === 'InvoiceInCard');
Porque :key="$router.path" no sirve?
No, no detecta el cambio.
@ -155,3 +153,3 @@
<QTd />
<QTd>
{{ toCurrency(getTotal(rows, 'amount'), currency) }}
{{ getTotal(rows, 'amount', { currency: 'default' }) }}
En invoiceInDueDay, los parametros son los mismo, declaramos valores por defecto?
No entiendo lo que quieres decir.
El composable getTotal, tiene un 3er parametro vacio.
A lo mejor podriamos reemplzar opts={} por opts = {currency: 'default'}
No siempre se que quiere devolver el total en valor monetario. A veces solo se quiere hacer la suma de cantidades.
New commits pushed, approval review dismissed automatically according to repository settings
@ -0,0 +18,4 @@
<template #form-inputs="{ data, validate }">
<VnRow>
<VnInput
:label="`${t('Code')}`"
global.code
@ -0,0 +33,4 @@
</VnRow>
<VnRow>
<VnInput
:label="`${t('Description')}`"
globals.description
Me falta tooltip del
@ -2,18 +2,17 @@
import { ref, computed } from 'vue';
En el boton + de la tabla me falta un tooltip
@ -226,2 +179,2 @@
class="cursor-pointer"
size="xs"
<template #form>
<CreateNewExpenseForm
Juraría que el botón del mas puede llevar tooltip
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.
Me espero a mañana para fusionar, debido a los problemas que han habido hoy con las transacciones
He encontrado una alternativa al onBeforeRouter
De todas maneras, tambien lo cambio porque dijiomos de mover la misma logica al composable
@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