Skip to content

Conversation

@ifere
Copy link
Contributor

@ifere ifere commented Jan 21, 2021

What was wrong?

getTransactionCount needed to be moved to snake case.

Related to Issue #1429

How was it fixed?

Fix not complete only initialized pull request to avoid overlapping work

Todo:

Cute Animal Picture

Put a link to a cute animal picture inside the parenthesis-->

@kclowes kclowes force-pushed the eth-get-transaction-count-at-snake-case branch from f010bab to ab5a1ec Compare January 29, 2021 23:46
@kclowes
Copy link
Collaborator

kclowes commented Jan 29, 2021

I caught this up with master, and removed the fixture you added so your parity integration tests should pass now! I'll do a more thorough review on Monday! Thanks!

@kclowes kclowes force-pushed the eth-get-transaction-count-at-snake-case branch from e5e0ee5 to 7dba288 Compare February 10, 2021 22:46
@kclowes kclowes force-pushed the eth-get-transaction-count-at-snake-case branch from 7dba288 to 298ee94 Compare February 10, 2021 22:58
Copy link
Collaborator

@kclowes kclowes left a comment

Choose a reason for hiding this comment

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

Thanks @ifere!

@kclowes kclowes merged commit 125c6ae into ethereum:master Feb 10, 2021
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.

2 participants