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

Implementation of eth_get_transaction_count function #983

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

Gerson2102
Copy link

@Gerson2102 Gerson2102 commented Sep 26, 2024

This PR include changes on the following file crates/contracts/src/kakarot_core/eth_rpc.cairo. I'm doing the implementation of the eth_get_transaction_count function. The issue is not done yet.

Pull Request type

Normal

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no API changes)
  • Build-related changes
  • Documentation content changes
  • Other (please describe):

Resolves: #942

Does this introduce a breaking change?

  • Yes
  • No

This change is Reviewable

@Gerson2102
Copy link
Author

I was checking in the codebase for more places where we can now use the eth_get_transaction_count but I didn't find any to be honest. Let me know if there is any that is missing. @enitrat

@Gerson2102
Copy link
Author

Gerson2102 commented Sep 29, 2024

I added one test. But I was thinking about change the nonce and check it again. But I dont really know how to do it... Maybe using set_nonce()?

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.

feat: implement eth_get_transaction_count
2 participants