refs #5835 migrateInvoiceIn #110
Labels
No Milestone
No Assignees
4 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix-front#110
Loading…
Reference in New Issue
No description provided.
Delete Branch "5835-migrateInvoiceIn"
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?
5835-migrateInvoiceInto WIP 5835-migrateInvoiceInWIP 5835-migrateInvoiceInto 5835-migrateInvoiceIn@ -0,0 +49,4 @@
async function setEditDms(dmsId) {
const { data } = await axios.get(`Dms/${dmsId}`);
dms.value = {
lo miramos
@ -0,0 +117,4 @@
}
}
async function create() {
Yo movería la lógica de create y save a funciones constantes porque estás duplicando código que hacen lo mismo
@ -0,0 +85,4 @@
async function edit() {
try {
if (!dms.value.companyId) throw Error(t(`The company can't be empty`));
Incluso te diría que en vez de tantos ifs, haría un objeto de validaciones, donde la key es el campo y el value es el mensaje a mostrar.
Para la que aplica en edit, usuaria la bandera que hemos comentado
5835-migrateInvoiceInto refs #5835 migrateInvoiceInAhora lo miramos en tu pc como ha quedado
@ -0,0 +5,4 @@
const token = session.getToken();
export async function downloadFile(dmsId) {
let appUrl = await getUrl('', 'lilium');
confirma con @alexm que lilum en el código a piñón es correcto
Lo veo bien. Si se llega a repetir mucho, se podria llegar a hacer getUrlLilium o alho
New commits pushed, approval review dismissed automatically according to repository settings
New commits pushed, approval review dismissed automatically according to repository settings
New commits pushed, approval review dismissed automatically according to repository settings
New commits pushed, approval review dismissed automatically according to repository settings
New commits pushed, approval review dismissed automatically according to repository settings
@ -394,0 +395,4 @@
pageTitles: {
invoiceIns: 'Fact. recibidas',
list: 'Listado',
createInvoiceOut: 'Crear fact. recibida',
invoice Out es factura emitida
pero dejalo en Crear factura
@ -394,0 +401,4 @@
vat: 'IVA',
dueDay: 'Vencimiento',
intrastat: 'Intrastat',
log: 'Registros de auditoría',
aquí creo que en salix es historico, puede ser?
En salix se llama histórico pero en Lilium le han puesto ese nombre en las otras secciones
@ -0,0 +208,4 @@
<!--Basic Data-->
<QCard class="vn-one">
<QCardSection class="q-pa-none">
<a class="header" :href="`#/invoice-in/${entityId}/basic-data`">
4 llamadas al mismo formato de URL, podríamos tenerlo en un método con un argumento?
New commits pushed, approval review dismissed automatically according to repository settings
New commits pushed, approval review dismissed automatically according to repository settings
New commits pushed, approval review dismissed automatically according to repository settings
@ -0,0 +187,4 @@
intrastatTotals.value = { ...getIntrastatTotals(entity.invoiceInIntrastat) };
}
function taxRate(taxableBase, rate) {
Revisamos si añadiendo valores por defecto podemos quitar líneas
@ -87,3 +88,4 @@
padding: 15px;
> .q-card.vn-one {
width: 350px;
tamaño fijo? Es correcto?
Según hablé con Juan sí, lo ideal es que tengan el mismo tamaño.
@ -404,0 +405,4 @@
pageTitles: {
invoiceIns: 'Fact. recibidas',
list: 'Listado',
createInvoiceOut: 'Crear fact. recibida',
invoice out es emitida, mira si esta mal la clave o el valor
@ -0,0 +452,4 @@
option-value="id"
option-label="code"
@input-value="companiesRef.fetch()"
:rules="[(val) => val || t('Required field')]"
Esta rule esta muchas veces en el código y es la misma en todos los casos.
Se podría sacar del HTML y definir en el script setup?
@ -0,0 +363,4 @@
v-model="data.booked"
clearable
clear-icon="close"
mask="####-##-##"
Los valores de las propiedades de mask, podrían ser globales en el componente?
@ -0,0 +222,4 @@
>
<template #menu="{ entity }">
<QItem
v-if="!entity.isBooked && hasAny(['administrative'])"
Igual que has hecho isAgricultural, podrías hacer para isAdministrative?
@ -0,0 +305,4 @@
<style lang="scss" scoped>
.q-dialog {
.q-card {
width: 35em;
tiene sentido tener un width y un max-width de igual tamaño?
Se puede dar el caso que el valor de la qcard sea mas pequeño¿?
@ -0,0 +67,4 @@
const isNotEuro = (code) => code != 'EUR';
async function insert() {
await axios.post('/InvoiceInDueDays/new ', { id: Number(invoiceId) });
TIP: es igual poner Number(invoiceId) que +invoiceId
@ -0,0 +99,4 @@
<QInput
v-model="row.dueDated"
mask="date"
placeholder="yyyy/mm/dd"
Quizás debería ser una valor constante
@ -0,0 +116,4 @@
{
name: 'code',
label: 'invoiceIn.summary.code',
field: (row) => {
({intrastat}) en vez de row
@ -0,0 +112,4 @@
const isNotEuro = (code) => code != 'EUR';
function taxRate(invoiceInTax, sageTaxTypeId) {
Lo revisamos
@ -0,0 +332,4 @@
<QInput
:label="t('Taxable base')"
:class="{
'no-pointer-events': isNotEuro(
En InvoiceInDueDay, la condición estaba al revés, correcto?