-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Fix read contract and change tx count #6309
Conversation
…bility issue with smart contract interaction
@@ -962,6 +962,17 @@ defmodule Explorer.Chain do | |||
Repo.aggregate(to_address_query, :count, :hash, timeout: :infinity) | |||
end | |||
|
|||
@spec address_hash_to_transaction_count(Hash.Address.t()) :: non_neg_integer() | |||
def address_hash_to_transaction_count(address_hash) do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The naming is incorrect. Since it accounts all transactions on address (to/from), I would suggest changing the naming of the function to address_hash_all_transaction_count
@@ -2447,17 +2458,7 @@ defmodule Explorer.Chain do | |||
|
|||
@spec address_to_transaction_count(Address.t()) :: non_neg_integer() | |||
def address_to_transaction_count(address) do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is worth changing the naming of the function since now it accounts all transactions on the address.
Some functions still should be re-named (see review comments)
Close #5738
Close #5518
Changelog
Checklist for your Pull Request (PR)
CHANGELOG.md
with this PRmaster
in the Version column. Changes will be reflected in this table: https://docs.blockscout.com/for-developers/information-and-settings/env-variables.