-
Notifications
You must be signed in to change notification settings - Fork 192
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #1469 +/- ##
===========================================
- Coverage 45.87% 45.82% -0.06%
===========================================
Files 274 274
Lines 6855 6865 +10
Branches 1343 1345 +2
===========================================
+ Hits 3145 3146 +1
- Misses 3548 3557 +9
Partials 162 162
Continue to review full report at Codecov.
|
.then(network => { | ||
console.log('fetchNetworkConfig') | ||
this.form.version = network.version.toString() | ||
this.$error('Address Version was missing - please check your wallets') |
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.
Maybe move this to the locale file and remove debugging.
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.
Yes you're right 🤔 will do it a bit later on. Thanks 👌
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.
Done @luciorubeens
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
oops |
Summary
The network
version
value wasn't being saved in previous wallet versions. This is a patch to re-populate theversion
from the network.Checklist