Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

richapatel1510
Copy link
Contributor

What/Why?

Convert class component AddressForm into function component.

Replacing class components with function components eliminates the need for traditional lifecycle methods and enables full adoption of React 18 features like hooks and concurrent rendering.

This modernization aligns with React’s roadmap and ensures greater compatibility with future updates.

Rollout/Rollback

Revert this PR.

Testing

  • CI checks

@richapatel1510 richapatel1510 marked this pull request as ready for review August 27, 2025 06:57
@richapatel1510 richapatel1510 requested a review from a team as a code owner August 27, 2025 06:57
@richapatel1510 richapatel1510 requested a review from bc-peng August 29, 2025 02:03
@richapatel1510 richapatel1510 force-pushed the CHECKOUT-9454-AddressForm branch from a0b34c4 to 76464bb Compare September 3, 2025 01:15
? getBillingCountries()
: getShippingCountries())
|| EMPTY_ARRAY;
const googleMapsApiKey = config?.checkoutSettings.googleMapsApiKey || '';
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this a formatting issue?

bc-peng
bc-peng previously approved these changes Sep 5, 2025
@richapatel1510 richapatel1510 merged commit 9008331 into master Sep 12, 2025
11 checks passed
@richapatel1510 richapatel1510 deleted the CHECKOUT-9454-AddressForm branch September 12, 2025 01:15
@bc-launchbay
Copy link

checkout-js deployed to Integration US
success 20250912013004 by Launchbay

@bc-launchbay
Copy link

checkout-js deployed to Staging US
success 20250912013004 by Launchbay

Copy link

⚡️ Lighthouse results

🖥️ Desktop:

Category Score
🟢 Performance 92
🟢 Accessibility 91
🟠 Best practices 78

📱 Mobile:

Category Score
🟠 Performance 59
🟢 Accessibility 94
🟠 Best practices 78

@bc-launchbay
Copy link

checkout-js deployed to Production US
success 20250912013004 by Launchbay

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.

4 participants