-
Notifications
You must be signed in to change notification settings - Fork 49
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
feature(wallet-mobile): new tx review for collateral #3683
Conversation
…tx-review-token-details
d15e481
to
e017a6d
Compare
e017a6d
to
b7f710f
Compare
onSuccessChanged(onSuccess) | ||
onErrorChanged(onError) |
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.
onError
and onSuccess
variables will be removed when product designs common screens
operationsChanged([<Operation key="0" />]) | ||
onSuccessChanged(onSuccess) | ||
onErrorChanged(onError) |
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.
onError
and onSuccess
variables will be removed when product designs common screens
The operations are very specific for each screen so I have decided to make them dependent on the screen that uses tx review
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
Description / Change(s) / Related issue(s)
Add new tx review for collateral funnel
Ticket
YV-36