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

WIP: fix: use salix for myTpvTransaction_end #127

Draft
ldragan wants to merge 2 commits from ldragan/hedera-web:taro/tpv into beta
pull from: ldragan/hedera-web:taro/tpv
merge into: verdnatura:beta
Conversation 0 Commits 2 Files Changed 1 +7 -3
ldragan commented 2025-03-19 01:03:24 +00:00
Member
Copy Link

TODO: probar este cambio (cómo?)

Intenté probar esta llamada insertando data a mano en la DB, pero no logré crear data falsa que pase ciertas validaciones. Por otro lado, me daría más tranquilidad hacer una prueba "real" — siguiendo un flujo, como lo haría un usuario, y no insertando data falsa a mano en la DB.

TODO: probar este cambio (cómo?) Intenté probar esta llamada insertando data a mano en la DB, pero no logré crear data falsa que pase ciertas validaciones. Por otro lado, me daría más tranquilidad hacer una prueba "real" — siguiendo un flujo, como lo haría un usuario, y no insertando data falsa a mano en la DB.
ldragan added 1 commit 2025-03-19 01:03:25 +00:00
gitea/hedera-web/pipeline/pr-beta This commit looks good Details
1c3684c8c7 fix: use salix for myTpvTransaction_end
ldragan reviewed 2025-03-19 01:04:03 +00:00
src/stores/tpv.js Outdated
@ -15,0 +14,4 @@
await api.post('applications/myTpvTransaction_end/execute-proc', new URLSearchParams({
schema: 'hedera',
params: `[${order}, ${status}]`,
ldragan commented 2025-03-19 01:04:03 +00:00
Author
Member
Copy Link

Si order o status no son numéricos, esto debería incluir comillas.

Si `order` o `status` no son numéricos, esto debería incluir comillas.
jsegarra commented 2025-03-21 13:07:02 +00:00
Member
Copy Link

creo que con JSON.stringify se debería resolver.
Al menos así lo hemos hecho en Lilium

creo que con JSON.stringify se debería resolver. Al menos así lo hemos hecho en Lilium
❤️ 1
ldragan added 1 commit 2025-03-21 03:03:42 +00:00
gitea/hedera-web/pipeline/pr-beta This commit looks good Details
2733b13953 refactor: URLSearchParams -> JSON
All checks were successful
gitea/hedera-web/pipeline/pr-beta This commit looks good
Required
Details
This pull request is marked as a work in progress.
This branch is out-of-date with the base branch
Sign in to join this conversation.
Reviewers
No reviewers
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#127
No description provided.
Delete Branch "ldragan/hedera-web:taro/tpv"

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?