-
Notifications
You must be signed in to change notification settings - Fork 110
frontend: show bitrefill address in a blocking dialog #3552
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
frontend: show bitrefill address in a blocking dialog #3552
Conversation
5afc79a
to
10d8d4c
Compare
@Beerosagos ready for review. Added a few more commits:
|
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.
tACK with a couple of nits 🙏
// Ensure expected payment method matches account | ||
if (coinMapping[account.coinCode] !== paymentMethod) { | ||
alertUser(t('unknownError', { errorMessage: 'Payment method mismatch' })); | ||
setPendingPayment(false); |
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.
I think we need a return too here
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.
good catch 🙏
const [agreedTerms, setAgreedTerms] = useState(false); | ||
|
||
const [pendingPayment, setPendingPayment] = useState<boolean>(false); | ||
const [verifyPaymentRequest, setVerifyPaymentRequest] = useState<TTxProposalResult & { address: string } | false>(); |
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.
Initialize to false?
So the user can confirm the payment on the device.
e68fd76
to
b092422
Compare
So the user can confirm the payment on the device.
Before asking for reviews, here is a check list of the most common things you might need to consider: