-
Notifications
You must be signed in to change notification settings - Fork 974
Conversation
Codecov Report
@@ Coverage Diff @@
## master #12593 +/- ##
=========================================
+ Coverage 56.08% 56.19% +0.1%
=========================================
Files 278 278
Lines 27078 27090 +12
Branches 4385 4387 +2
=========================================
+ Hits 15187 15223 +36
+ Misses 11891 11867 -24
|
8d04baa
to
44e2ddd
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.
++ Tests were as expected
app/browser/api/ledger.js
Outdated
const newAddress = result.getIn(['properties', 'wallet', 'addresses', 'BAT']) | ||
const oldAddress = ledgerState.getInfoProps(state).getIn(['addresses', 'BAT']) | ||
|
||
console.log(newAddress, oldAddress) |
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.
whoops- looks like you left a console log in there 😄
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.
console removed
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.
comment left- otherwise looks good! 😄 👍
oh- and needs rebase 😛
44e2ddd
to
5a49b57
Compare
Resolves brave#12592 Auditors: Test Plan:
5a49b57
to
25fec63
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.
++
Fixes qr code when it is new
Fixes qr code when it is new
Submitter Checklist:
git rebase -i
to squash commits (if needed).Resolves #12592
Auditors:
Test Plan: defined in #12592
Reviewer Checklist:
Tests