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

#7750 - first-beta-review #95

Merged
jsegarra merged 26 commits from wbuezas/hedera-web-mindshore:first-beta-review into beta 2024-12-09 23:18:07 +00:00
Conversation 4 Commits 26 Files Changed 27 +280 -209
wbuezas commented 2024-12-05 21:40:26 +00:00
Contributor
Copy Link
No description provided.
wbuezas added 25 commits 2024-12-05 21:40:27 +00:00
gitea/hedera-web/pipeline/pr-beta There was a failure building this commit Details
09866f5b41 Add promise all in catalog
wbuezas requested review from jsegarra 2024-12-05 21:41:50 +00:00
jsegarra changed title from first-beta-review to #7750 - first-beta-review 2024-12-09 12:05:00 +00:00
jsegarra requested changes 2024-12-09 12:30:04 +00:00
Dismissed
jsegarra left a comment
Member
Copy Link

reemplazar data-testid por data-cy

reemplazar data-testid por data-cy
package.json
@ -10,7 +10,6 @@
"type": "git",
"url": "https://git.verdnatura.es/hedera-web"
},
jsegarra commented 2024-12-09 12:20:41 +00:00
Member
Copy Link

porque lo eliminamos?

porque lo eliminamos?
wbuezas commented 2024-12-09 12:33:14 +00:00
Author
Contributor
Copy Link

Porque estaba duplicado

Porque estaba duplicado
👍 1 😕 1
jsegarra marked this conversation as resolved
src/pages/Ecomerce/CatalogView.vue
@ -748,2 +763,4 @@
const filtered = res.results[1].data.filter(item => item.category);
itemSubcategories.value = filtered.map(i => i.category);
} catch (error) {
console.error('Error getting subcategories:', error);
jsegarra commented 2024-12-09 12:25:04 +00:00
Member
Copy Link

En lilium hemos quitado los trycatch porque si la peticion falla, el usuario no se entera.
Ya tenemos el controlador de axios, no?

En lilium hemos quitado los trycatch porque si la peticion falla, el usuario no se entera. Ya tenemos el controlador de axios, no?
wbuezas commented 2024-12-09 12:33:53 +00:00
Author
Contributor
Copy Link

Si, tenemos el controlador de axios

Si, tenemos el controlador de axios
src/pages/Ecomerce/CheckoutView.vue
@ -182,0 +185,4 @@
const found = agencies.value.find(
agency => agency.id === defaultValues.value.defaultAgencyFk
);
jsegarra commented 2024-12-09 12:26:38 +00:00
Member
Copy Link

Duda, si no se encuentra no debería mostrar mensaje de error al usuario?
Cambiar found por agency

Duda, si no se encuentra no debería mostrar mensaje de error al usuario? Cambiar found por agency
wbuezas commented 2024-12-09 14:33:45 +00:00
Author
Contributor
Copy Link

No creo que se deba mostrar un error por esto, supongo que por algo el hedera viejo no lo hacía

No creo que se deba mostrar un error por esto, supongo que por algo el hedera viejo no lo hacía
src/stores/app.js Outdated
@ -190,0 +174,4 @@
},
localeDates() {
const { messages, locale } = i18n.global;
return {
jsegarra commented 2024-12-09 12:29:51 +00:00
Member
Copy Link

dale una vuelta porque esto se puede simplificar mucho

dale una vuelta porque esto se puede simplificar mucho
wbuezas commented 2024-12-09 14:32:45 +00:00
Author
Contributor
Copy Link

Simplificado

Simplificado
❤️ 1
jsegarra marked this conversation as resolved
jsegarra commented 2024-12-09 12:35:25 +00:00
Member
Copy Link

@wbuezas a falta de tener el entorno correctamente puesto en mi pc, cuando puedas, pasa los test y adjunta el reporte como adjunto de un comentario, porfa

@wbuezas a falta de tener el entorno correctamente puesto en mi pc, cuando puedas, pasa los test y adjunta el reporte como adjunto de un comentario, porfa
wbuezas added 1 commit 2024-12-09 14:31:04 +00:00
gitea/hedera-web/pipeline/pr-beta This commit looks good Details
dedb8a73a5 Resolve conflicts and simplyfy localeDates
wbuezas commented 2024-12-09 19:28:29 +00:00
Author
Contributor
Copy Link

@wbuezas a falta de tener el entorno correctamente puesto en mi pc, cuando puedas, pasa los test y adjunta el reporte como adjunto de un comentario, porfa

gitea no me deja subir el reporte por el formato de archivo, lo paso al grupo de rocket:

https://chat.verdnatura.es/group/SalixMindshore?msg=dJGRRrQAhMXfNL9Mc

> @wbuezas a falta de tener el entorno correctamente puesto en mi pc, cuando puedas, pasa los test y adjunta el reporte como adjunto de un comentario, porfa gitea no me deja subir el reporte por el formato de archivo, lo paso al grupo de rocket: https://chat.verdnatura.es/group/SalixMindshore?msg=dJGRRrQAhMXfNL9Mc
jsegarra commented 2024-12-09 21:28:03 +00:00
Member
Copy Link

@wbuezas a falta de tener el entorno correctamente puesto en mi pc, cuando puedas, pasa los test y adjunta el reporte como adjunto de un comentario, porfa

gitea no me deja subir el reporte por el formato de archivo, lo paso al grupo de rocket:

https://chat.verdnatura.es/group/SalixMindshore?msg=dJGRRrQAhMXfNL9Mc

Si lo zipeas si que te deja

> > @wbuezas a falta de tener el entorno correctamente puesto en mi pc, cuando puedas, pasa los test y adjunta el reporte como adjunto de un comentario, porfa > > gitea no me deja subir el reporte por el formato de archivo, lo paso al grupo de rocket: > > https://chat.verdnatura.es/group/SalixMindshore?msg=dJGRRrQAhMXfNL9Mc Si lo zipeas si que te deja
jsegarra commented 2024-12-09 21:29:22 +00:00
Member
Copy Link

el reporte

el reporte
report_first_hedera_review.zip
366 KiB
jsegarra approved these changes 2024-12-09 23:18:01 +00:00
jsegarra merged commit 0f1bde3b77 into beta 2024-12-09 23:18:07 +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#95
No description provided.
Delete Branch "wbuezas/hedera-web-mindshore:first-beta-review"

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?