verdnatura/hedera-web
verdnatura
/
hedera-web
0
0
Fork 1
Code Issues Pull Requests 1 Releases Wiki Activity

Pedidos stepper #80

Merged
jsegarra merged 16 commits from wbuezas/hedera-web-mindshore:feature/PedidosStepper into 4922-vueMigration 2024-09-03 14:06:02 +00:00
Conversation 1 Commits 16 Files Changed 19 +966 -283
wbuezas commented 2024-08-30 13:21:33 +00:00
Contributor
Copy Link
No description provided.
wbuezas added 3 commits 2024-08-30 13:21:33 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration There was a failure building this commit Details
cb2c9871cc Stepper
wbuezas requested review from jsegarra 2024-08-30 13:21:42 +00:00
wbuezas added 1 commit 2024-08-30 21:06:46 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
c77c2e6648 Update branch
jsegarra requested changes 2024-08-30 21:36:13 +00:00
Dismissed
src/pages/Ecomerce/CheckoutView.vue
@ -2,0 +195,4 @@
warehouses.value = results[1].data;
if (!warehouses.value || !warehouses.value.length) {
notify(t('noWarehousesAvailableForDate'), 'negative');
jsegarra commented 2024-08-30 21:33:49 +00:00
Member
Copy Link

Se que no está en dev, pero esta notificación se puede mostrar y que no cambie de paso, asi te permite escoger una dirección diferente sin hacer un click extra. Aunque lo ideal, si el mensaje dice que para esa fecha no hay, que te lleve directamente al paso de fecha.
Lo anotaria como propuesta de mejora

Se que no está en dev, pero esta notificación se puede mostrar y que no cambie de paso, asi te permite escoger una dirección diferente sin hacer un click extra. Aunque lo ideal, si el mensaje dice que para esa fecha no hay, que te lleve directamente al paso de fecha. Lo anotaria como propuesta de mejora
wbuezas commented 2024-09-02 15:52:09 +00:00
Author
Contributor
Copy Link

Lo podemos poner en la lista de propuestas para mejoras

Lo podemos poner en la lista de propuestas para mejoras
jsegarra commented 2024-09-02 20:16:46 +00:00
Member
Copy Link

Anotado

Anotado
jsegarra marked this conversation as resolved
src/pages/Ecomerce/CheckoutView.vue Outdated
@ -2,0 +348,4 @@
<span class="text-h6 text-bold tex q-mb-md text-center">
{{ t('orderDateDelivery') }}
</span>
<QDate
jsegarra commented 2024-08-30 21:30:19 +00:00
Member
Copy Link

La fecha está en inglés

La fecha está en inglés
wbuezas commented 2024-09-02 15:43:43 +00:00
Author
Contributor
Copy Link

Corregido.

Commit: 020e0afc96

Corregido. Commit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/020e0afc96427ca1bf6db03dba58e4b1f38b8b27
jsegarra marked this conversation as resolved
src/pages/Ecomerce/CheckoutView.vue Outdated
@ -2,0 +355,4 @@
/>
</div>
<!-- Address step -->
<QList
jsegarra commented 2024-08-30 21:29:09 +00:00
Member
Copy Link

Este step no tiene titulo?

Este step no tiene titulo?
wbuezas commented 2024-09-02 15:44:09 +00:00
Author
Contributor
Copy Link

Corregido.

Commit: f2bd3c2fa6

Corregido. Commit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/f2bd3c2fa6f6db469205c772612d1b1b28cde459
jsegarra marked this conversation as resolved
src/pages/Ecomerce/CheckoutView.vue Outdated
@ -2,0 +356,4 @@
</div>
<!-- Address step -->
<QList
v-if="step.name === 'address'"
jsegarra commented 2024-08-30 21:34:54 +00:00
Member
Copy Link

Con el segundo metodo, he podido avvanzar sin seleccionar una dirección. Esto en los otros pasos no ocurre.

Con el segundo metodo, he podido avvanzar sin seleccionar una dirección. Esto en los otros pasos no ocurre.
wbuezas commented 2024-09-02 15:50:28 +00:00
Author
Contributor
Copy Link

Validación agregada.

Commit: 3a21292030

Validación agregada. Commit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/3a21292030bc7b091de789e8771cd69114e09602
jsegarra marked this conversation as resolved
src/pages/Ecomerce/CheckoutView.vue Outdated
@ -2,0 +380,4 @@
</QItem>
</QList>
<!-- Agency step (AGENCY) -->
<div
jsegarra commented 2024-08-30 21:29:55 +00:00
Member
Copy Link

Si selecciono "Recibir en mi tienda", me aparece el mismo titulo que seleccionar fecha

Si selecciono "Recibir en mi tienda", me aparece el mismo titulo que seleccionar fecha
wbuezas commented 2024-09-02 15:50:49 +00:00
Author
Contributor
Copy Link

Corregido.

Commit: 6423ecfb05

Corregido. Commit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/6423ecfb05a63b0cd6b637e378e75da733591d5b
jsegarra marked this conversation as resolved
src/pages/Ecomerce/CheckoutView.vue Outdated
@ -2,0 +429,4 @@
flat
color="primary"
@click="onPreviousStep(stepIndex)"
label="Back"
jsegarra commented 2024-08-30 21:28:29 +00:00
Member
Copy Link

traducir

traducir
wbuezas commented 2024-09-02 15:51:18 +00:00
Author
Contributor
Copy Link

Traducido.

Commit: 95e23c05fa

Traducido. Commit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/95e23c05fa8dbdb329fa72b7edd034faf0656ff5
jsegarra marked this conversation as resolved
src/pages/Ecomerce/CheckoutView.vue Outdated
@ -2,0 +436,4 @@
<QBtn
@click="onNextStep(stepIndex)"
color="primary"
label="next"
jsegarra commented 2024-08-30 21:28:36 +00:00
Member
Copy Link

traducir

traducir
wbuezas commented 2024-09-02 15:51:26 +00:00
Author
Contributor
Copy Link

Traducido.

Commit: 95e23c05fa

Traducido. Commit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/95e23c05fa8dbdb329fa72b7edd034faf0656ff5
jsegarra marked this conversation as resolved
src/pages/Ecomerce/CheckoutView.vue Outdated
@ -2,0 +437,4 @@
@click="onNextStep(stepIndex)"
color="primary"
label="next"
/>
jsegarra commented 2024-08-30 21:36:10 +00:00
Member
Copy Link

La label del botón en el ultimo paso debería ser confirmar, finalizar, guardar, confirmar con Javi

La label del botón en el ultimo paso debería ser confirmar, finalizar, guardar, confirmar con Javi
wbuezas commented 2024-09-02 15:51:51 +00:00
Author
Contributor
Copy Link

Agregado.

Commit: 8e0f09cc0f

Agregado. Commit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/8e0f09cc0f73afda885f28991fb40689a1bf9e2e
❤️ 1
jsegarra marked this conversation as resolved
jsegarra reviewed 2024-08-30 21:37:21 +00:00
src/pages/Ecomerce/CheckoutView.vue Outdated
@ -2,0 +349,4 @@
{{ t('orderDateDelivery') }}
</span>
<QDate
v-model="orderForm.date"
jsegarra commented 2024-08-30 21:37:21 +00:00
Member
Copy Link

Empieza en domingo, a nivel de empresa podría servir, pero decara a un usuario final, puede llevar a confusión

Empieza en domingo, a nivel de empresa podría servir, pero decara a un usuario final, puede llevar a confusión
wbuezas commented 2024-09-02 15:52:25 +00:00
Author
Contributor
Copy Link

Agregado.

Commit: 975495113d

Agregado. Commit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/975495113d2d7b2c5257d3f9b3d9f7e17f1cb7a8
❤️ 1
jsegarra marked this conversation as resolved
wbuezas added 7 commits 2024-09-02 15:36:31 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
975495113d Set monday as first day of week
wbuezas added 1 commit 2024-09-02 15:39:16 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
7026e3416f Remove fetching of default order method
wbuezas added 1 commit 2024-09-02 19:29:27 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
723a977ecd Small fix
jsegarra reviewed 2024-09-02 20:19:21 +00:00
src/pages/Ecomerce/Catalog.vue
@ -41,0 +42,4 @@
rounded
no-caps
:to="{
name: 'checkout',
jsegarra commented 2024-09-02 20:19:21 +00:00
Member
Copy Link

Duda: en Lilium usamos UpperCamelCase, mientras que aquí usamos camelCase.
Ambos estándares son aceptables, pero como verias estandarizar en favor de "Lilium"

Duda: en Lilium usamos UpperCamelCase, mientras que aquí usamos camelCase. Ambos estándares son aceptables, pero como verias estandarizar en favor de "Lilium"
wbuezas commented 2024-09-02 20:36:35 +00:00
Author
Contributor
Copy Link

Si, coincido en estandarizar, pero en el proyecto ya lo venían haciendo así.
Consideré en hacer el cambio pero conllevaria también cambiar todos los namings de las traducciones de las secciones ya que aparentemente el name de la ruta se usa como título de la sección.

Si, coincido en estandarizar, pero en el proyecto ya lo venían haciendo así. Consideré en hacer el cambio pero conllevaria también cambiar todos los namings de las traducciones de las secciones ya que aparentemente el `name` de la ruta se usa como título de la sección.
👎 1
jsegarra reviewed 2024-09-02 20:24:02 +00:00
src/pages/Ecomerce/CheckoutView.vue
@ -2,0 +119,4 @@
return address.street;
});
const confirmPlaceText = computed(() => {
jsegarra commented 2024-09-02 20:24:02 +00:00
Member
Copy Link

Ufff...si vamos bien de tiempo, arreglamos esta funcion:

  1. El método find hace lo mismo, solo cambia el origen de datos
  2. La lógica del return hace lo mismo, solo cambia la etiqueta que quremos traducir
Ufff...si vamos bien de tiempo, arreglamos esta funcion: 1. El método find hace lo mismo, solo cambia el origen de datos 2. La lógica del return hace lo mismo, solo cambia la etiqueta que quremos traducir
jsegarra reviewed 2024-09-02 20:28:27 +00:00
src/stores/app.js Outdated
@ -66,2 +97,3 @@
return true;
} catch (err) {
console.error('Error checking order', err);
if (err.exception === 'Vn.Lib.UserError') {
jsegarra commented 2024-09-02 20:28:27 +00:00
Member
Copy Link

What?

What?
wbuezas commented 2024-09-02 20:31:16 +00:00
Author
Contributor
Copy Link

Literal lo copie y pegue del viejo hedera, no se ni lo que es jajaja

Literal lo copie y pegue del viejo hedera, no se ni lo que es jajaja
😆 1
jsegarra reviewed 2024-09-02 20:28:46 +00:00
src/stores/app.js Outdated
@ -72,3 +131,3 @@
if (this.basketOrderId !== orderId) {
localStorage.setItem('hederaBasket', orderId);
this.basketOrderId = orderId;
localStorage.setItem('hederaBasket', orderId);
jsegarra commented 2024-09-02 20:28:46 +00:00
Member
Copy Link

Porque no usamos pinia?

Porque no usamos pinia?
wbuezas added 1 commit 2024-09-02 20:53:52 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
ba2ded5c48 Add contracted prop for narrow windows
jsegarra commented 2024-09-02 20:59:24 +00:00
Member
Copy Link

Se corrige el fallo responsive para dispositivos de 4.7"
Antes: image
Después
image

Se corrige el fallo responsive para dispositivos de 4.7" Antes: ![image](/attachments/eee367dc-9291-488d-a0a9-69a4c8d51b55) Después ![image](/attachments/7e7fe61c-5616-45e9-981d-b76f3b816f72)
image.png
32 KiB
image.png
10 KiB
image.png
32 KiB
image.png
wbuezas added 1 commit 2024-09-02 21:27:31 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
6631be401b Add minimal to QDate when is mobile
wbuezas added 1 commit 2024-09-03 11:17:34 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
6bfa5b9715 Small changes
jsegarra approved these changes 2024-09-03 11:25:14 +00:00
jsegarra merged commit 05568280f3 into 4922-vueMigration 2024-09-03 14:06:02 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
jsegarra
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
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#80
No description provided.
Delete Branch "wbuezas/hedera-web-mindshore:feature/PedidosStepper"

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?