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

feature/Address-view-refactor #123

Merged
jsegarra merged 6 commits from wbuezas/hedera-web-mindshore:feature/Address-view-refactor into beta 2025-03-21 14:40:28 +00:00
Conversation 2 Commits 6 Files Changed 17 +491 -156
wbuezas commented 2025-03-10 13:13:46 +00:00
Member
Copy Link
No description provided.
wbuezas added 3 commits 2025-03-10 13:13:47 +00:00
gitea/hedera-web/pipeline/pr-beta This commit looks good Details
ae45ef91e1 Last changes
wbuezas requested review from jsegarra 2025-03-10 13:13:54 +00:00
jsegarra requested changes 2025-03-12 12:30:42 +00:00
Dismissed
package.json
@ -60,3 +60,3 @@
"db": "cd ../salix && gulp docker",
"cy:open": "npm run db && cypress open",
"test:e2e": "npm run db && cypress run",
"test:e2e": "npm run db && cypress run --headed --config video=false",
jsegarra commented 2025-03-12 11:54:39 +00:00
Member
Copy Link

esto es necesario?

esto es necesario?
wbuezas commented 2025-03-13 13:15:45 +00:00
Author
Member
Copy Link

Si, ya que aparentemente hay un bug en cypresss que en la version terminal fallan los tests.

Si, ya que aparentemente hay un bug en cypresss que en la version terminal fallan los tests.
quasar.config.js
@ -24,3 +24,3 @@
// --> boot files are part of "main.js"
// https://v2.quasar.dev/quasar-cli-webpack/boot-files
boot: ['i18n', 'axios', 'vnDate', 'error-handler', 'app'],
boot: ['i18n', 'axios', 'vnDate', 'app'],
jsegarra commented 2025-03-12 11:54:53 +00:00
Member
Copy Link

porque lo quitamos?

porque lo quitamos?
wbuezas commented 2025-03-13 13:16:20 +00:00
Author
Member
Copy Link

Porque es innecesario, ya hacemos error handling en el archivo de axios, por esto aparecian duplicados los notify de errores.

Porque es innecesario, ya hacemos error handling en el archivo de axios, por esto aparecian duplicados los notify de errores.
👍 1
wbuezas marked this conversation as resolved
src/pages/Account/AddressDetails.vue Outdated
@ -123,3 +151,4 @@
:label="t('province')"
:options="provincesOptions"
data-cy="addressFormProvince"
:required="true"
jsegarra commented 2025-03-12 11:57:27 +00:00
Member
Copy Link

y si le pasamos required tal cual, no deberia ir?
O es porque no estamos comporbando que la key de required exista en los $attrs?

y si le pasamos required tal cual, no deberia ir? O es porque no estamos comporbando que la key de required exista en los $attrs?
wbuezas commented 2025-03-14 17:42:42 +00:00
Author
Member
Copy Link

Recuerdo que en un momento en lilium se hacia algo asi por algo en especial. Pero parece que aca pasando solo required funciona igual

Commit: 5e3387fb6d

Recuerdo que en un momento en lilium se hacia algo asi por algo en especial. Pero parece que aca pasando solo `required` funciona igual Commit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/5e3387fb6d1af7ebc97b730e41cbbb6a3db58600
wbuezas added 1 commit 2025-03-14 17:42:09 +00:00
gitea/hedera-web/pipeline/pr-beta This commit looks good Details
5e3387fb6d Modify required attr
jsegarra commented 2025-03-16 09:27:14 +00:00
Member
Copy Link

Hay que revisar porque están fallando 7 test en esta rama mientras que en beta fallan 5
He subido de version de cypress(como hemos hecho en lilium) pero fallan mas, asi que nos quedamos con la actual.

Hay que revisar porque están fallando 7 test en esta rama mientras que en beta fallan 5 He subido de version de cypress(como hemos hecho en lilium) pero fallan mas, asi que nos quedamos con la actual.
wbuezas added 1 commit 2025-03-19 14:51:02 +00:00
gitea/hedera-web/pipeline/pr-beta This commit looks good Details
92dc8998bf Input change
wbuezas commented 2025-03-19 16:44:20 +00:00
Author
Member
Copy Link

Hay que revisar porque están fallando 7 test en esta rama mientras que en beta fallan 5
He subido de version de cypress(como hemos hecho en lilium) pero fallan mas, asi que nos quedamos con la actual.

Ahora no fallan los tests, adjunto reporte

> Hay que revisar porque están fallando 7 test en esta rama mientras que en beta fallan 5 > He subido de version de cypress(como hemos hecho en lilium) pero fallan mas, asi que nos quedamos con la actual. Ahora no fallan los tests, adjunto reporte
test report.zip
368 KiB
jsegarra approved these changes 2025-03-21 14:07:55 +00:00
jsegarra added 1 commit 2025-03-21 14:08:05 +00:00
gitea/hedera-web/pipeline/pr-beta This commit looks good Details
73feb098fe Merge branch 'beta' into feature/Address-view-refactor
jsegarra merged commit 1138274243 into beta 2025-03-21 14:40:28 +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#123
No description provided.
Delete Branch "wbuezas/hedera-web-mindshore:feature/Address-view-refactor"

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?