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

Update copy on crypto payment dialog #1938

Merged
merged 6 commits into from
Feb 14, 2022
Merged

Conversation

FSM1
Copy link
Contributor

@FSM1 FSM1 commented Feb 11, 2022

closes #1926


Submission checklist:

Layout

  • Change looks good in the desktop web ui
  • Change looks good in the mobile web ui

Theme

  • Components / elements inspected in light mode
  • Components / elements inspected in dark mode

@render
Copy link

render bot commented Feb 11, 2022

@render
Copy link

render bot commented Feb 11, 2022

@render
Copy link

render bot commented Feb 11, 2022

@github-actions github-actions bot added the Type: Feature Added to PRs to identify that the change is a new feature. label Feb 11, 2022
Copy link
Collaborator

@Tbaut Tbaut left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 setting back the indentation

@FSM1 FSM1 requested review from asnaith, tanmoyAtb and Tbaut February 11, 2022 15:40
@FSM1 FSM1 self-assigned this Feb 11, 2022
@FSM1 FSM1 added the Status: Review Needed 👀 Added to PRs when they need more review label Feb 11, 2022
@FSM1 FSM1 changed the title fix copy Update copy on crypto payment dialog Feb 11, 2022
@Tbaut Tbaut enabled auto-merge (squash) February 11, 2022 17:12
@Tbaut Tbaut merged commit 9a508ff into dev Feb 14, 2022
@Tbaut Tbaut deleted the feat/crypto-full-payment-1926 branch February 14, 2022 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Review Needed 👀 Added to PRs when they need more review Type: Feature Added to PRs to identify that the change is a new feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Include full payment reminder on the "Pay with crypto" modal
4 participants