-
Notifications
You must be signed in to change notification settings - Fork 862
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
Adds additional balance check #2091
Conversation
d32a943
to
9e7e29e
Compare
9e7e29e
to
cba09ed
Compare
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.
test plan works, changes look fine 👍
std::unique_ptr<ledger::WalletInfo> wallet_info) { | ||
TriggerOnWalletProperties(result, std::move(wallet_info)); | ||
if (wallet_info && wallet_info->balance_ > 0) { |
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.
logic from TriggerOnWalletProperties
was moved in this function
Adds additional balance check
@NejcZdovc are there any tests for this change? |
@bsclifton not particular, just existing ones. Contribution flow will be covered with this tests #1804 |
Merge pull request #2091 from brave/balance-check
Resolves brave/brave-browser#3897
Submitter Checklist:
npm test brave_unit_tests && npm test brave_browser_tests && npm run test-security
) onnpm run lint
)git rebase master
(if needed).git rebase -i
to squash commits (if needed).Test Plan:
Reviewer Checklist: