-
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-1584] offDeviceMnemonic
factor source kind
#532
Conversation
Sources/Features/SettingsFeature/ManageFactorSourcesFeature/ManageFactorSources.swift
Show resolved
Hide resolved
offDeviceMnemonic
factor source kind
Sources/Clients/FactorSourcesClient/FactorSourcesClient+Interface.swift
Outdated
Show resolved
Hide resolved
Sources/Clients/FactorSourcesClientLive/FactorSourcesClient+Live.swift
Outdated
Show resolved
Hide resolved
Sources/Features/ImportMnemonic/OffDeviceMnemonicInfo/OffDeviceMnemonicInfo.swift
Outdated
Show resolved
Hide resolved
...ttingsFeature/ImportFromOlympiaLegacyWallet/Coordinator/ImportOlympiaWalletCoordinator.swift
Outdated
Show resolved
Hide resolved
Sources/Features/SettingsFeature/ManageFactorSourcesFeature/ManageFactorSources.swift
Outdated
Show resolved
Hide resolved
Labeled("Device ID", value: device.id) | ||
Labeled("Creation date", value: device.date.ISO8601Format()) | ||
Labeled("Device", value: device.description.rawValue) |
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.
Please add FIXME: Strings in every file where strings are added...
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.
This is Debug Inspect Profile
will never ever reach end user :)
case let .offDeviceMnemonicInfoPrompt(.presented(.delegate(.done(label, description, mnemonicWithPassphrase)))): | ||
state.offDeviceMnemonicInfoPrompt = nil | ||
precondition(state.saveInProfileKind == .offDevice) | ||
return .run { send in |
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.
this looks like a job for task
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.
@kugel3 hit refresh :D
already changed into a task :)
Sources/Features/ImportMnemonic/OffDeviceMnemonicInfo/OffDeviceMnemonicInfo+View.swift
Show resolved
Hide resolved
WithViewStore(store, observe: \.viewState, send: { .view($0) }) { viewStore in | ||
VStack { | ||
AppTextField( | ||
primaryHeading: "Tell a story", |
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.
FIXME: Strings please
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.
sure, but not even sure we wanna have this screen.... I just invented it and asking for input from Matt about it... but putting FIXMEs there :)
Sources/Features/ImportMnemonic/OffDeviceMnemonicInfo/OffDeviceMnemonicInfo.swift
Outdated
Show resolved
Hide resolved
...ttingsFeature/ImportFromOlympiaLegacyWallet/Coordinator/ImportOlympiaWalletCoordinator.swift
Outdated
Show resolved
Hide resolved
@@ -35,15 +35,20 @@ extension ManageFactorSources { | |||
} | |||
} | |||
} | |||
Button("Import Mnemonic source") { | |||
viewStore.send(.importMnemonicButtonTapped) | |||
Button("Import Olympia mnemonic") { |
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.
Fixme
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.
This is debug screen only, should probably be renamed as such...
Jira ticket: ABW-1584 , ticket 0 in MFA epic.
Signing
Signing with this factor will be implemented in a subsequent PR.
Screenshot
Video
Movie in Slack