-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use receipts from TransactionStatus
#1644
Labels
feat
Issue is a feature
Comments
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This was referenced Mar 28, 2024
This was referenced Apr 11, 2024
4 tasks
This was referenced Aug 9, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Discussed in #1588
Originally posted by nedsalk December 15, 2023
Post
fuel-core v0.22.0
, the node now returns receipts with aTransactionStatus
, so we can use them right away instead of running another fetch. From my understanding, this will only become relevant after #1374 is merged, because we're currently using thegetTransactionWithReceipts
endpoint.The way to go about implementing this is to use the
TransactionRequest
sent to the node inprovider.sendTransaction
because it contains the same transaction as the one that's fetched from the node inside ofTransactionResponse
viagetTransactionWithReceipts
, only that after fetching it from the node the receipts are there as well.Because the receipts can now be received from the subscription directly, the same transaction summary can be generated based off of the
TransactionRequest
and the receipts received from the subscription. A working implementation can be found on this branch.The text was updated successfully, but these errors were encountered: