Skip to content
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: handle loadedAddresses field in tx meta responses #27065

Merged
merged 1 commit into from
Aug 10, 2022

Conversation

jstarry
Copy link
Member

@jstarry jstarry commented Aug 10, 2022

Problem

The loadedAddresses field is returned with transaction metadata but is currently ignored

Summary of Changes

Add support for detecting and validating the loadedAddresses field in tx metadata responses

Fixes #

@jstarry jstarry added the automerge Merge this Pull Request automatically once CI passes label Aug 10, 2022
@codecov
Copy link

codecov bot commented Aug 10, 2022

Codecov Report

Merging #27065 (61c7de7) into master (1165a7f) will decrease coverage by 4.3%.
The diff coverage is n/a.

❗ Current head 61c7de7 differs from pull request most recent head e9e6098. Consider uploading reports for the commit e9e6098 to get more accurate results

@@             Coverage Diff             @@
##           master   #27065       +/-   ##
===========================================
- Coverage    81.9%    77.5%     -4.4%     
===========================================
  Files         631       42      -589     
  Lines      174252     2466   -171786     
  Branches        0      352      +352     
===========================================
- Hits       142728     1912   -140816     
+ Misses      31524      424    -31100     
- Partials        0      130      +130     

@mergify mergify bot merged commit 369b6b1 into solana-labs:master Aug 10, 2022
@jstarry jstarry deleted the web3/meta-loaded-addresses branch August 10, 2022 20:53
xiangzhu70 pushed a commit to xiangzhu70/solana that referenced this pull request Aug 17, 2022
…s#27065)

feat: handle loadedAddresses field in tx meta responses
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge this Pull Request automatically once CI passes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants