vercel / nextjs-subscription-payments

Clone, deploy, and fully customize a SaaS subscription application with Next.js.
https://subscription-payments.vercel.app/
MIT License
6.23k stars 1.27k forks source link

copyBillingDetailsToCustomer - Why return on !name, !phone, or !address? #146

Open oalexdoda opened 1 year ago

oalexdoda commented 1 year ago

Hey,

I noticed that there's a copyBillingDetailsToCustomer() function that runs on createOrRetrieveCustomer.

Any reason why it returns when there's a missing name, phone or address?

What if a customer doesn't add their phone? Or address? Any reason why their billing details shouldn't be updated?

Thanks!

thorwebdev commented 1 year ago

Not really, probably can be removed. Do you want to open a PR?