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

Remove duplicate of balances_params_to_address_params function #5480

Merged

Conversation

vbaranov
Copy link
Member

@vbaranov vbaranov commented Apr 19, 2022

Motivation

balances_params_to_address_params/1 is defined in 2 fetchers (coin balance, contract code).

Changelog

Leave function only in con balance fetcher and re-use it in contract code fetcher.

Checklist for your Pull Request (PR)

@vbaranov vbaranov force-pushed the vb-remove-double-balances_params_to_address_params-func branch from 0bab9a2 to 39f1306 Compare April 19, 2022 09:22
@vbaranov vbaranov force-pushed the vb-remove-double-balances_params_to_address_params-func branch from 39f1306 to 4212a56 Compare April 19, 2022 09:42
@vbaranov vbaranov merged commit 1b815df into master Apr 19, 2022
@vbaranov vbaranov deleted the vb-remove-double-balances_params_to_address_params-func branch April 19, 2022 09:43
sidhujag pushed a commit to syscoin/blockscout that referenced this pull request May 11, 2022
…alances_params_to_address_params-func

Remove duplicate of balances_params_to_address_params function
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.

1 participant