Add `requiresBillingInfo` prop to the ResourceAddress component #815

marcomontalbano · 2024-10-25T18:09:00Z

What I did

This PR adds/edits a few things to the ResourceAddress component:

💥 Breaking Change - I renamed the prop resource to address.
I added a new prop called requiresBillingInfo. When true, it forces billing_info as a required field.
I removed unnecessary re-renders.

…s component

… billing_info validation

netlify · 2024-10-25T18:09:16Z

Name	Link
🔨 Latest commit	`38da238`
🔍 Latest deploy log	https://app.netlify.com/sites/commercelayer-app-elements/deploys/671bdebec2f4320008dc4fc5
😎 Deploy Preview	https://deploy-preview-815--commercelayer-app-elements.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

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

marcomontalbano added 3 commits October 25, 2024 18:00

fix: avoid unnecessary re-renders on ResourceAddress and Overlay

c2df9bb

chore!: rename the prop 'resource' to 'address' in the ResourceAddres…

7583549

…s component

feat!: add new prop 'requiresBillingInfo' to ResourceAddress to force…

38da238

… billing_info validation

marcomontalbano requested review from gciotola and pfferrari October 25, 2024 18:09

gciotola approved these changes Oct 28, 2024

View reviewed changes

marcomontalbano merged commit 3122944 into main Oct 28, 2024
5 checks passed

marcomontalbano deleted the resource-address-new-requiresBillingInfo-prop branch October 28, 2024 08:49

marcomontalbano self-assigned this Oct 28, 2024

marcomontalbano added breaking-change enhancement New feature or request labels Oct 28, 2024