-
Notifications
You must be signed in to change notification settings - Fork 985
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
[#9034] Turn off autosuggestion on seedphrase confirmation input #9051
Conversation
Pull Request Checklist
|
@@ -14,7 +14,7 @@ | |||
(dissoc merged-styles :background-color) | |||
merged-styles))) | |||
|
|||
(defn text-input-with-label [{:keys [label content error style height container text editable] :as props :or {editable true}}] | |||
(defn text-input-with-label [{:keys [label content error style height container text editable keyboard-type] :as props :or {editable true}}] |
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.
(defn text-input-with-label [{:keys [label content error style height container text editable keyboard-type] :as props :or {editable true}}] | |
(defn text-input-with-label | |
[{:keys [label content error style height container text editable keyboard-type] | |
:as props | |
:or {editable true}}] |
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 break the line it is too long
Jenkins Builds
|
98% of end-end tests have passed
Failed tests (1)Click to expand
Passed tests (46)Click to expand |
@flexsurfer please include my suggestion in the commit. Also I tested and the suggestion is gone in the profile but they are still there when you are creating an account with keycard and you are asked to confirm seed phrase |
e516544
to
2ac0ca0
Compare
Signed-off-by: Andrey Shovkoplyas <motor4ik@gmail.com>
2ac0ca0
to
6a73257
Compare
why you merged the suggestion are still there in onboarding :( |
@yenda my bad, hadn't seen your comment :( |
fixes #9034