-
Notifications
You must be signed in to change notification settings - Fork 922
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
Rename Direct Keymanager to Imported #7549
Conversation
@@ -43,7 +43,7 @@ func (k Kind) String() string { | |||
switch k { | |||
case Derived: | |||
return "derived" | |||
case Direct: | |||
case Imported: | |||
return "direct" |
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.
Not changed because it will break people's 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.
If theres anytime to do this isn't it for the beta release?
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.
Looks good. I didn't check if it needs to align with the docs portal. Please make sure that it algns. Thank you
Codecov Report
@@ Coverage Diff @@
## master #7549 +/- ##
==========================================
+ Coverage 61.67% 61.81% +0.14%
==========================================
Files 424 424
Lines 29897 29897
==========================================
+ Hits 18439 18482 +43
+ Misses 8491 8447 -44
- Partials 2967 2968 +1 |
This is part of #7515. This PR Removes all instances of non-HD or direct wallets and renames them to
imported
to reflect that they must be initialized by importing keystores from an external source