-
Notifications
You must be signed in to change notification settings - Fork 9
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
[ABW-1301] Ledger genesis factor instances #451
Conversation
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.
Awesome!
Added some minor comments, as the part of the UI as I understand it is a subject for later improvements.
Sources/Clients/ImportLegacyWalletClient/ImportLegacyWalletClient+Live.swift
Show resolved
Hide resolved
Sources/Clients/LedgerHardwareWalletClient/LedgerHardwareWalletClient+Live.swift
Show resolved
Hide resolved
@@ -17,6 +17,13 @@ public enum EntityKind: | |||
|
|||
/// Used by Persona | |||
case identity = 618 | |||
|
|||
public var entityType: any EntityProtocol.Type { |
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 typeOfEntity would better? it is a bit confusing to have entityType on EntityKind.
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.
which is why I we call all kinds "Kind", so that we can call a type, "type", it matches the type, but I dont feel too strongly about it.
Jira ticket: ABW-1301
Description
Adds the ability to create Account and Persona with a Ledger as genesis factor source.
Video
https://rdxworks.slack.com/archives/C031A0V1A1W/p1682073839199249
PR submission checklist