Conversation
andreabadesso
left a comment
There was a problem hiding this comment.
I think it would be great to be more explicit on how the specific events from the reliable integrations feed are going to be handled, like NEW_VERTEX_ACCEPTED and VERTEX_METADATA_CHANGED
projects/wallet-lib-client-for-event-queue/0001-event-queue-connection-design.md
Show resolved
Hide resolved
projects/wallet-lib-client-for-event-queue/0001-event-queue-connection-design.md
Show resolved
Hide resolved
projects/wallet-lib-client-for-event-queue/0001-event-queue-connection-design.md
Show resolved
Hide resolved
projects/wallet-lib-client-for-event-queue/0001-event-queue-connection-design.md
Show resolved
Hide resolved
There was a problem hiding this comment.
The fullnode already sends it decoded, the documentation might not have been updated because this was a change I requested, here is an example input:
{
"tx_id": "00000000000000a7880484eb5f059e7857973d7c4d131a9bd59389afe6b0dfc6",
"index": 0,
"spent_output": {
"value": 800,
"token_data": 0,
"script": "dqkUtlo02wdlNsUJx3OADdSG/uaHGnWIrA==",
"decoded": {
"type": "P2PKH",
"address": "HP9KRutTVPPf5h8jZ3BMwfhkJoZpD1c7if",
"timelock": null
}
}
}
|
|
||
| #### Events | ||
|
|
||
| - `new-tx` and `update-tx` |
There was a problem hiding this comment.
Semi-unrelated to line number, but remember to add a ping/pong mechanism
There was a problem hiding this comment.
The document of the event queue and the implementation did not have ping/pong anywhere but from what I understand it comes with the websocket lib we use, so I'll add the ping/pong to the design
There was a problem hiding this comment.
Autobahn answers the PING frame with a PONG and also disconnects when no messages are received in a window. It also doesn't send a PING, so the implementation must be done in the client
There was a problem hiding this comment.
We also need to assert that the stream_id is the same from the last sync
There was a problem hiding this comment.
| If the `voided_by` field is the same we can iterate on the outputs and calculate | |
| If the `voided_by` field is empty we can iterate on the outputs and calculate |
There was a problem hiding this comment.
Used null instead of empty but to the same effect.
Uh oh!
There was an error while loading. Please reload this page.