-
Notifications
You must be signed in to change notification settings - Fork 137
Implements cross-platform Backup/Recovery modal #274
Conversation
@@ -12,12 +12,13 @@ export interface Props { | |||
readOnly?: boolean | |||
placeholder?: string | |||
disabled?: boolean | |||
fieldError?: boolean |
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.
Per designs, we need to be able to highlight the text area error red in case of a problem
cc: @jenn-rhim
recoveryKey: event.target.value | ||
}) | ||
} | ||
|
||
onRestore = (key?: string) => { | ||
key = typeof key === 'string' ? key : this.state.recoveryKey | ||
this.setState({ |
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.
Per design, we want to retain the recovery key in case of a problem
cc: @jenn-rhim
@@ -33,10 +37,12 @@ export interface Props { | |||
error?: React.ReactNode | |||
id?: string | |||
testId?: string | |||
funds?: string |
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.
We need to be able to produce a special warning for restore if the user has funds in their wallet.
What should be fed in this prop from assembly is a formatted string of the users BAT balance. Ex: '55 BAT'
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.
unable to check the code right now but since CI is good I'm approving to unblock work. we can re-visit in a follow-up if something is not ok
Live storybook link: https://brave-ui-ubfmotahgl.now.sh
Preview:
(Slight differences in design vs shared components)