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

[Wallet][Refactor] Updating ancient getbalanceXXX methods to a lambda call. #1056

Merged

Conversation

furszy
Copy link

@furszy furszy commented Oct 25, 2019

Refactored and cleaned every getBalanceXXX method that was duplicating the same code over and over inside the wallet to a loop over txs lambda based function.

Copy link

@Warrows Warrows left a comment

Choose a reason for hiding this comment

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

ACK 7001b05

@random-zebra random-zebra added this to the 4.0.0 milestone Oct 27, 2019
Copy link

@random-zebra random-zebra left a comment

Choose a reason for hiding this comment

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

Very nice 👍 ACK 7001b05

random-zebra added a commit that referenced this pull request Oct 27, 2019
…s to a lambda call.

7001b05 [Wallet][Refactor] Getbalance and similar methods lambda refactored. (furszy)

Pull request description:

  Refactored and cleaned every `getBalanceXXX` method that was duplicating the same code over and over inside the wallet to a loop over txs lambda based function.

ACKs for top commit:
  Warrows:
    ACK 7001b05
  random-zebra:
    Very nice 👍 ACK 7001b05

Tree-SHA512: b43355b08a2a3c6118357f10076a57cbc8c21f83f3dfb86f0366eabdfe9d6706b9686d18593d74f3e5bfab0dff4936de6b12eaffbf2a13e438d3c0b8eb4019c2
@random-zebra random-zebra merged commit 7001b05 into PIVX-Project:master Oct 27, 2019
@furszy furszy deleted the 2019_wallet_balances_lambda branch November 29, 2022 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants