Vistas sección pedidos #77

Merged
jsegarra merged 14 commits from wbuezas/hedera-web-mindshore:feature/Pedidos into 4922-vueMigration 2024-08-16 06:52:22 +00:00
Contributor
  • Creación de componentes CardList, TicketDetails, InvoicesView.vue, OrdersView.vue, PendingOrders.vue, TicketView.vue
  • Se agregó la prop: rounded a VnImg.vue
  • Se crearon e importaron los archivos de traducciones faltantes con sus respectivas traducciones
  • Creación de actions relacionadas a Orders en el store de app
  • Ahora el header muestra el nombre de las secciones traducidos
  • Soy partidario del pascal case en el naming de las rutas, pero se estaba usando camel case, así que decidí respetarlo. Más que nada proque el name de la ruta luego es utilizado en los archivos de traducciones para darle el título a la sección
- Creación de componentes `CardList`, `TicketDetails`, `InvoicesView.vue`, `OrdersView.vue`, `PendingOrders.vue`, `TicketView.vue` - Se agregó la `prop: rounded` a `VnImg.vue` - Se crearon e importaron los archivos de traducciones faltantes con sus respectivas traducciones - Creación de actions relacionadas a `Orders` en el store de `app` - Ahora el header muestra el nombre de las secciones traducidos - Soy partidario del `pascal case` en el naming de las rutas, pero se estaba usando `camel case`, así que decidí respetarlo. Más que nada proque el name de la ruta luego es utilizado en los archivos de traducciones para darle el título a la sección
wbuezas added 3 commits 2024-08-06 12:21:23 +00:00
wbuezas requested review from jsegarra 2024-08-06 12:21:32 +00:00
Member

@wbuezas
Revisamos AddressList porque no guarda, ni muestra mensaje de error al eliminar una fila

@wbuezas Revisamos AddressList porque no guarda, ni muestra mensaje de error al eliminar una fila
jsegarra requested changes 2024-08-13 08:10:59 +00:00
Dismissed
jsegarra left a comment
Member

Revisamos la subtoolbar

Revisamos la subtoolbar
src/App.vue Outdated
@ -2,10 +2,10 @@
<router-view />
Member

Movemos la etiqueta script setup al principio del archivo

Movemos la etiqueta script setup al principio del archivo
Author
Contributor

Modificado.

Commit: b66c47955c

Modificado. Commit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/b66c47955c6bdbd5c92137450b4d5bff3f67d3d5
jsegarra marked this conversation as resolved
@ -0,0 +22,4 @@
const lineSubtotal = line => {
const discount = line.discount;
return lineDiscountSubtotal(line) * ((100 - discount) / 100);
Member

porque no haces line.discount en vez de crear una variable?

porque no haces line.discount en vez de crear una variable?
Author
Contributor

Modificado.

Commit: a0fc1cfc07

Modificado. Commit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/a0fc1cfc070cd790923b7067580aabb77be7a0fd
jsegarra marked this conversation as resolved
@ -1,73 +1,87 @@
import { date as qdate, format } from 'quasar'
const { pad } = format
import { i18n } from 'src/boot/i18n';
Member

TODO: revisar si vale la pena crear un fichero para cada filter

TODO: revisar si vale la pena crear un fichero para cada filter
Author
Contributor

Esto lo pensamos mejor y si vemos que vale la pena lo metemos para la proxima PR

Esto lo pensamos mejor y si vemos que vale la pena lo metemos para la proxima PR
jsegarra marked this conversation as resolved
jsegarra requested changes 2024-08-14 10:12:08 +00:00
Dismissed
jsegarra left a comment
Member

Revisar tooltip de los botones del módulo de pedidos

Revisar tooltip de los botones del módulo de pedidos
jsegarra requested changes 2024-08-14 10:36:19 +00:00
Dismissed
@ -0,0 +40,4 @@
const params = new URLSearchParams({
access_token: userStore.token
});
return `/api/InvoiceOuts/${id}/download?${params}`;
Member

Si esto devuelve un PDF, podriamos aprovechar para user usePrintservice(Lilium)

Si esto devuelve un PDF, podriamos aprovechar para user usePrintservice(Lilium)
Author
Contributor

Agregado.

Commit: f2c8b90324

Agregado. Commit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/f2c8b90324b0f051734e7752cd73a3a36c531015
jsegarra marked this conversation as resolved
@ -0,0 +82,4 @@
<div class="vn-w-sm">
<QCard>
<!-- -->
<QTable
Member

Podemos aplicar el estilo de la tabla de Agencias?

Podemos aplicar el estilo de la tabla de Agencias?
Author
Contributor

Cree el componente VnTable y lo aplique en las vistas PackagesView e InvoicesView.

Commit: f36eb1bd88

Cree el componente `VnTable` y lo aplique en las vistas `PackagesView` e `InvoicesView`. Commit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/f36eb1bd88c019265f9c789f1f1db3ae369b5891
Member

Ok,
TODO: para la próxima PR añadir la función de ordenar la columna en InvoicesView porque para packages si que está

Ok, TODO: para la próxima PR añadir la función de ordenar la columna en InvoicesView porque para packages si que está
@ -0,0 +137,4 @@
requestTheInvoiceToComercial: Request the invoice to your salesperson
es-ES:
noInvoicesFound: No se han encontrado facturas
serial: Serie
Member

Factura en vez de serie. O ya estaba este literal?

Factura en vez de serie. O ya estaba este literal?
Author
Contributor

Modificado.

Commit: 7c96106faa

Modificado. Commit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/7c96106faa22e3ccdbf6aa993b254cff8b8157f4
jsegarra marked this conversation as resolved
Member

El selector de años no está en producción, correcto?

El selector de años no está en producción, correcto?
jsegarra requested changes 2024-08-14 10:52:38 +00:00
Dismissed
@ -0,0 +21,4 @@
label: t('issued'),
field: 'issued',
align: 'left',
format: val => qdate.formatDate(val, 'D MMM YYYY')
Member

Revisamos el formateo de fechas porque aparece en ingles

Revisamos el formateo de fechas porque aparece en ingles
Author
Contributor

Util de formateo de fechas con traducciones creada.

Commit: 887ee8aea4

Util de formateo de fechas con traducciones creada. Commit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/887ee8aea47247e720fd6afd86468d69a4f49596
jsegarra marked this conversation as resolved
@ -0,0 +32,4 @@
defaultAmountStr = amount;
}
amount = prompt(t('amountToPay'), defaultAmountStr);
Member

Podemos hacer un modal en vez del prompt?
Se que es lo que hay en producción, pero ahora tenemos Vue

Podemos hacer un modal en vez del prompt? Se que es lo que hay en producción, pero ahora tenemos Vue
Author
Contributor

Prompt reemplazado.

Commit: 44627dbc8a

Prompt reemplazado. Commit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/44627dbc8a4b69debd80491a4a191d2edce6c6ec
Member

Apuntamos para la próxima PR aplicar la condicion de solo numeros, porque si está vacio o hay letras te dice en rojo lo siguiente: "La cantidad debe ser un número positivo e inferior o igual al importe pendiente"

Apuntamos para la próxima PR aplicar la condicion de solo numeros, porque si está vacio o hay letras te dice en rojo lo siguiente: "La cantidad debe ser un número positivo e inferior o igual al importe pendiente"
wbuezas added 1 commit 2024-08-14 11:37:15 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
b66c47955c
Move script tag to the start of the file
wbuezas added 1 commit 2024-08-14 11:39:59 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
a0fc1cfc07
use line.discount directly
wbuezas added 1 commit 2024-08-14 11:47:32 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
7c96106faa
Change serial column name to invoice
wbuezas added 1 commit 2024-08-14 11:54:25 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
b728ecaf29
Add vn date
wbuezas added 1 commit 2024-08-14 12:08:55 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
f36eb1bd88
Create VnTable and use it
wbuezas added 1 commit 2024-08-14 12:10:02 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
e0f55f8ca3
Change class casing
wbuezas added 1 commit 2024-08-14 12:31:51 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
f2c8b90324
Create print service and fix slot in VnTable
wbuezas added 1 commit 2024-08-14 13:56:17 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
44627dbc8a
Replace prompt with VnConfirm
wbuezas added 1 commit 2024-08-14 14:28:23 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
887ee8aea4
Create date format util with translations
wbuezas added 1 commit 2024-08-14 15:16:54 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
594b17b4ab
Fix teleport submit problem
Author
Contributor

@wbuezas
Revisamos AddressList porque no guarda, ni muestra mensaje de error al eliminar una fila

Problema de que no guardaba solucionado.
Decidimos dejar el mostrar mensaje de error en el interceptor de jApi para la próxima PR.

Commit: 594b17b4ab

> @wbuezas > Revisamos AddressList porque no guarda, ni muestra mensaje de error al eliminar una fila Problema de que no guardaba solucionado. Decidimos dejar el mostrar mensaje de error en el interceptor de `jApi` para la próxima PR. Commit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/594b17b4abc8663a2a1534b331b6f4f1dfb668c0
wbuezas added 1 commit 2024-08-14 15:22:59 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
7e0c591026
Add tooltips
Author
Contributor

Revisar tooltip de los botones del módulo de pedidos

Tooltips de buttons añadidos.

Commit: 7e0c591026

> Revisar tooltip de los botones del módulo de pedidos Tooltips de buttons añadidos. Commit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/7e0c59102647071ee0a9566bf46d0afb3ac51a99
jsegarra reviewed 2024-08-16 06:52:06 +00:00
@ -0,0 +1,152 @@
<script setup>
Member

La ruta de este archivo debe estar en pages/Ecommerce porque sólo se usa en TicketView

La ruta de este archivo debe estar en pages/Ecommerce porque sólo se usa en TicketView
jsegarra approved these changes 2024-08-16 06:52:12 +00:00
jsegarra merged commit 5053a908f7 into 4922-vueMigration 2024-08-16 06:52:22 +00:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 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/hedera-web#77
No description provided.