-
Notifications
You must be signed in to change notification settings - Fork 2k
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
EditCardDetails: Refactor to a functional component #35237
Conversation
Here is how your PR affects size of JS and CSS bundles shipped to the user's browser: Sections (~22 bytes removed 📉 [gzipped])
Sections contain code specific for a given set of routes. Is downloaded and parsed only when a particular route is navigated to. Legend What is parsed and gzip size?Parsed Size: Uncompressed size of the JS and CSS files. This much code needs to be parsed and stored in memory. Generated by performance advisor bot at iscalypsofastyet.com. |
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.
👍 Looks good, tested well :)
In preparation to upgrade the `AddCardDetails` component to use Stripe Elements and Payment Intents (see previous work in #34848), this PR just reorganizes the component into a stateless functional component instead of a class. Previously this was done for `EditCardDetails` in #35237 which is nearly identical to this component. There wasn't much present that required a class in the first place; there was no state, and I think the redirect-on-invalid behavior is actually easier to understand happening on every render than using the lifecycle methods.
* AddCardDetails: Refactor as functional component In preparation to upgrade the `AddCardDetails` component to use Stripe Elements and Payment Intents (see previous work in #34848), this PR just reorganizes the component into a stateless functional component instead of a class. Previously this was done for `EditCardDetails` in #35237 which is nearly identical to this component. There wasn't much present that required a class in the first place; there was no state, and I think the redirect-on-invalid behavior is actually easier to understand happening on every render than using the lifecycle methods. * Fix big errors How did I miss these? Clearly I wasn't paying attention.
Changes proposed in this Pull Request
In preparation to upgrade the
EditCardDetails
component to use Stripe Elements and Payment Intents (see previous work in #34848), this PR just reorganizes the component into a stateless functional component instead of a class.There wasn't much present that required a class in the first place; there was no state, and I think the redirect-on-invalid behavior is actually easier to understand happening on every render than using the lifecycle methods.
Testing instructions
4242424242424242
) and submit the form. It will help to use a unique name for the cardholder name.