Skip to content

Conversation

@yoshidan
Copy link
Contributor

@yoshidan yoshidan commented Mar 7, 2025

Move account proof retrieval to ProveState to eliminate the need to run the old eth_getProof

Naohiro Yoshida added 4 commits March 7, 2025 16:06
Signed-off-by: Naohiro Yoshida <naohiro.yoshida@datachain.jp>
Signed-off-by: Naohiro Yoshida <naohiro.yoshida@datachain.jp>
Signed-off-by: Naohiro Yoshida <naohiro.yoshida@datachain.jp>
Signed-off-by: Naohiro Yoshida <naohiro.yoshida@datachain.jp>
@yoshidan yoshidan requested review from 3100 and removed request for 3100 March 8, 2025 04:31
Signed-off-by: Naohiro Yoshida <naohiro.yoshida@datachain.jp>
@yoshidan yoshidan requested a review from 3100 March 8, 2025 06:25
Signed-off-by: Naohiro Yoshida <naohiro.yoshida@datachain.jp>
@yoshidan yoshidan requested a review from bluele March 11, 2025 05:54
@yoshidan
Copy link
Contributor Author

@3100 The following commit combines storgeProof and accountProof acquisition into a single RPC.
d7f8660

Naohiro Yoshida added 2 commits March 11, 2025 21:51
Signed-off-by: Naohiro Yoshida <naohiro.yoshida@datachain.jp>
Signed-off-by: Naohiro Yoshida <naohiro.yoshida@datachain.jp>
Copy link
Member

@bluele bluele left a comment

Choose a reason for hiding this comment

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

LGTM👍

@yoshidan yoshidan merged commit 4970192 into v0.3.9_patch Mar 14, 2025
2 checks passed
@yoshidan yoshidan deleted the feature/move_account_verify branch March 14, 2025 08:35
yoshidan added a commit that referenced this pull request Mar 14, 2025
Move accountProof into ProveState (#53)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants