Skip to content
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

Include full payment reminder on the "Pay with crypto" modal #1926

Closed
asnaith opened this issue Feb 8, 2022 · 4 comments · Fixed by #1938
Closed

Include full payment reminder on the "Pay with crypto" modal #1926

asnaith opened this issue Feb 8, 2022 · 4 comments · Fixed by #1938
Assignees

Comments

@asnaith
Copy link
Contributor

asnaith commented Feb 8, 2022

Problem: Currently, if a user makes a partial crypto payment, we won’t act on it and it will stay in the crypto account. This will be handled by refund/credit flow in future but as of now this is the situation.

V1 Solution:

  • Add a reminder to the "Pay with crypto" modal informing the user that they must make a full payment when they are sending cyrpto

  • Update the currency label underneath QR code to use ETH rather than ethereum

Screen Shot 2022-02-08 at 10 07 04 AM

@Tbaut
Copy link
Collaborator

Tbaut commented Feb 8, 2022

We should not add yet another warning (there's already one after #1924) I'm for changing the copy eventually for the "Only send the exact amount of ETH to this address"

@asnaith
Copy link
Contributor Author

asnaith commented Feb 8, 2022

@Tbaut Sounds good, I was unaware of the other ticket. Perhaps all we need to discuss in planning is a copy change on this modal. Will keep it open for now.

@Tbaut
Copy link
Collaborator

Tbaut commented Feb 9, 2022

Absolutely, I change adapting the copy that is already there, on your screenshot, would make the most sense. Because the other warning I just added is about reimbursement rather than the payment itself.

@FSM1
Copy link
Contributor

FSM1 commented Feb 9, 2022

Hey team! Please add your planning poker estimate with ZenHub @tanmoyAtb @Tbaut

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants