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

feat: APP-202 buy credits step 2 #2418

Merged
merged 14 commits into from
Aug 8, 2024
Merged

feat: APP-202 buy credits step 2 #2418

merged 14 commits into from
Aug 8, 2024

Conversation

blushi
Copy link
Member

@blushi blushi commented Jul 30, 2024

Description

https://regennetwork.atlassian.net/browse/APP-202?atlOrigin=eyJpIjoiMGZmODlhMTc4YTY5NGYxNWJmZWM3MWE0MDYzNTNmNDAiLCJwIjoiaiJ9


Author Checklist

All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

I have...

  • provided a link to the relevant issue or specification
  • provided instructions on how to test
  • reviewed "Files changed" and left comments if necessary
  • confirmed all CI checks have passed

How to test

https://deploy-preview-2418--regen-storybook.netlify.app/?path=/story/marketplace-organisms-paymentinfoform--fiat-logged-out
https://deploy-preview-2418--regen-storybook.netlify.app/?path=/story/marketplace-organisms-paymentinfoform--fiat-logged-in-no-email
https://deploy-preview-2418--regen-storybook.netlify.app/?path=/story/marketplace-organisms-paymentinfoform--fiat-logged-in-with-email
https://deploy-preview-2418--regen-storybook.netlify.app/?path=/story/marketplace-organisms-paymentinfoform--fiat-logged-in-with-payment-method
https://deploy-preview-2418--regen-storybook.netlify.app/?path=/story/marketplace-organisms-paymentinfoform--crypto-no-email
https://deploy-preview-2418--regen-storybook.netlify.app/?path=/story/marketplace-organisms-paymentinfoform--crypto-with-email

Reviewers Checklist

All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items
.

I have...

  • confirmed all author checklist items have been addressed
  • reviewed code correctness and readability
  • verified React components follow DRY principles
  • reviewed documentation is accurate
  • reviewed tests
  • manually tested (if applicable)

Copy link

netlify bot commented Jul 30, 2024

Deploy Preview for regen-website ready!

Name Link
🔨 Latest commit b207fdc
🔍 Latest deploy log https://app.netlify.com/sites/regen-website/deploys/66b461514a3fde000818b795
😎 Deploy Preview https://deploy-preview-2418--regen-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Comment on lines 72 to 76
// react({
// babel: {
// plugins: ['macros'],
// },
// }),
Copy link
Member Author

@blushi blushi Jul 30, 2024

Choose a reason for hiding this comment

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

had to comment this temporarily, it's causing some issues running storybook locally
will need to investigate further cc/ @flagrede
reverted it back so the storybook build work

Comment on lines +3 to +4
// import { msg, Trans } from '@lingui/macro';
// import { useLingui } from '@lingui/react';
Copy link
Member Author

@blushi blushi Jul 30, 2024

Choose a reason for hiding this comment

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

TODO use lingui once issue running storybook locally is fixed
temporary fix is to do the following #2418 (comment)

@blushi blushi requested a review from a team July 30, 2024 09:44
@blushi blushi changed the title feat: APP-202 buy step 2 feat: APP-202 buy credits step 2 Jul 30, 2024
@blushi
Copy link
Member Author

blushi commented Jul 30, 2024

@erikalogie @clevinson see testing instructions

@erikalogie
Copy link
Collaborator

This looks great to me!

One small design detail on mobile:
Marketplace : Organisms : PaymentInfoForm - Fiat Logged Out ⋅ Storybook 2024-07-30 09-30-42

Also let's change the text under "Your name" to "This name will appear on the retirement certificate unless you choose to retire anonymously in the next step. It is also your user profile name." I think that is more clear.

@blushi
Copy link
Member Author

blushi commented Jul 31, 2024

@erikalogie done

@erikalogie
Copy link
Collaborator

LGTM!

@blushi blushi mentioned this pull request Aug 6, 2024
10 tasks
@blushi blushi requested review from r41ph and a team August 7, 2024 14:28
@blushi blushi enabled auto-merge (squash) August 8, 2024 06:11
@blushi blushi merged commit 30e0bcb into dev Aug 8, 2024
12 of 14 checks passed
@blushi blushi deleted the feat-APP-202-buy-step-2 branch August 8, 2024 06:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants