-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
feat: Add receipts to transaction status #1504
Merged
Merged
Changes from 15 commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
763a486
Add receipts to success status
bvrooman dbdf77a
Update schema
bvrooman 0393f9d
Merge branch 'master' into bvrooman/feat/add-receipts-to-tx-status
80117a0
Add receipts
bvrooman 9d57289
Update executor.rs
bvrooman ba4c731
Add txid to transaction status success and failed
bvrooman d5e42e0
update
bvrooman 2e228ce
Update receipt.rs
bvrooman 692071e
Add tests
bvrooman 4110de7
Cosmetics
bvrooman c24122b
Update CHANGELOG.md
bvrooman fd9e171
Update tests
bvrooman 9087c53
Update test utils
bvrooman 04ef877
Clippy
bvrooman bfde90f
Update txn_status_subscription.rs
bvrooman 171a025
Do not fail status when no receipts found
bvrooman 829161c
Merge branch 'master' into bvrooman/feat/add-receipts-to-tx-status
bvrooman 724859c
Use txid from query
bvrooman ce5b30d
Fix test compilation
bvrooman 04c4f8e
TransactionID on status
bvrooman 118979f
Update snapshots
bvrooman cec1f0c
Merge branch 'master' into bvrooman/feat/add-receipts-to-tx-status
0830825
Use constructor and remove unneeded query
bvrooman 51892af
Fix comment whitespace
bvrooman 1cc7142
Fix
bvrooman File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can remove it in this PR=)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I removed this query and replaced it with a block to read the receipts from the transaction status retrieved earlier in this function. This introduces an extra
clone()
because we now duplicate the receipts. This is okay for now - in the future, we can remove this function entirely because it is now redundant, and thesubmit_and_await_commit
endpoint will return receipts for all successful and failed transactions. I have added aTODO
inline to highlight this.