-
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
Remove CreateNewAccount from AssetTransferFlow #552
Remove CreateNewAccount from AssetTransferFlow #552
Conversation
Sources/Features/CreateAccount/Children/NameAccount/NameAccount+View.swift
Outdated
Show resolved
Hide resolved
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.
Shouldn't we just remove it, rather than have a flag? Will that boolean ever be dynamically set, even in the future?
Sources/Features/CreateAccount/Children/NameAccount/NameAccount+View.swift
Outdated
Show resolved
Hide resolved
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.
LGTM, just revert the text field change, and possible earlier PRs change to asciiCapable
keyboard type? I think I saw Umair reporting strange behaviour of textfield input ?
@kugel3 this is a very temporary fix, the CreateAccount flow should really just work and is disabled due to time constraints. Once we have the build sent for review, we need to fix the flow. |
Sure, so I would just comment it out, rather than adding a boolean property. |
This is used in other features also, like choose account during dApp Auth request. |
Hm, why does that use |
What do you mean? ChooseAccounts is a separate feature, which is used in TransferAccountList. |
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.
LGTM
Remove the ability to create a new Account form the AssetTransfer flow since it is very very buggy.