fix: make children optional

[Innei]

Make children optional. Because children are not required, as in the following usage, children is not passed in explicitly. We can ensure that types is compatible with the original version.

const baseContexts: JSX.Element[] = [
  <ThemeProvider key="themeProvider" />, // here, `children` is not passed.
  <JotaiStoreProvider key="jotaiStoreProvider" />,
]

export function WebAppProviders({ children }: PropsWithChildren) {
  return (
    <ProviderComposer contexts={baseContexts}>
      {children}
    </ProviderComposer>
  )
}

export const ProviderComposer: Component<{
  contexts: JSX.Element[]
}> = ({ contexts, children }) => {
  return contexts.reduceRight((kids: any, parent: any) => {
    return React.cloneElement(parent, { children: kids })
  }, children)
}

Thanks, @pacocoursey

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
next-themes-basic	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Mar 18, 2024 11:10am
next-themes-tailwind	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Mar 18, 2024 11:10am
next-themes-with-app-dir	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Mar 18, 2024 11:10am

[pacocoursey]

Fair enough!