next-learn icon indicating copy to clipboard operation
next-learn copied to clipboard

Apply a concrete type to the props of the InvoiceStatus component

Open mg5566 opened this issue 1 year ago • 2 comments

Hello

I was studying through next learn and came across the InvoiceStatus component.

It takes a status prop and I saw that it is defined as a string type. I'm using the status prop to handle conditional statements, but I think it would be better if it was declared as a concrete type.

mg5566 avatar Mar 23 '24 05:03 mg5566

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
next-learn-starter ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 23, 2024 5:48am
next-seo-starter ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 23, 2024 5:48am

vercel[bot] avatar Mar 23 '24 05:03 vercel[bot]

@mg5566 is attempting to deploy a commit to the Vercel Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] avatar Mar 23 '24 05:03 vercel[bot]

Hey! Thank you for the suggestion. I am okay with leaving it as is. But feel free to keep your version this way!

leerob avatar Jun 23 '24 22:06 leerob