Skip to content

Conversation

wjmelements
Copy link
Contributor

Reviewer @rvagg
Similar to #13274

Test Plan

Run local devnet miner and daemon.
Send transaction and await receipt, then kill miner.
Query eth_getTransactionCount of sender.
Before this change it returns zero.
After this change it returns nonzero

Changes

  • use LoadActorRaw to specify TipSetState cid

@wjmelements wjmelements requested a review from rvagg August 21, 2025 00:25
@github-project-automation github-project-automation bot moved this to 📌 Triage in FilOz Aug 21, 2025
wjmelements added a commit to FilOzone/filecoin-pay that referenced this pull request Aug 21, 2025
Copy link
Member

@rvagg rvagg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM sans changelog and title needs a colon after the (api) (bot should have complained at you about that so you must have found a way around).

@github-project-automation github-project-automation bot moved this from 📌 Triage to ✔️ Approved by reviewer in FilOz Aug 21, 2025
@wjmelements wjmelements enabled auto-merge (squash) August 21, 2025 16:51
@wjmelements wjmelements changed the title fix(api) eth_getTransactionCount operate on current tipset state fix(api): eth_getTransactionCount operate on current tipset state Aug 21, 2025
@wjmelements wjmelements merged commit f27a7e9 into master Aug 21, 2025
175 of 185 checks passed
@wjmelements wjmelements deleted the fix-eth_getTransactionCount-tipset branch August 21, 2025 18:57
@github-project-automation github-project-automation bot moved this from ✔️ Approved by reviewer to 🎉 Done in FilOz Aug 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 🎉 Done
Development

Successfully merging this pull request may close these issues.

2 participants