Skip to content

Fix react-confetti props type import#23

Open
alucard210319 wants to merge 1 commit into
kxxoling:masterfrom
alucard210319:fix/react-confetti-props-type
Open

Fix react-confetti props type import#23
alucard210319 wants to merge 1 commit into
kxxoling:masterfrom
alucard210319:fix/react-confetti-props-type

Conversation

@alucard210319

Copy link
Copy Markdown

Fixes #22.

Summary:

  • Stop importing the removed named Props type from react-confetti.
  • Derive the confetti options type from the Confetti component with React ComponentProps instead.
  • Use ReactElement as the return/state element type.

Why:
The reported build fails at src/components/Confetti.tsx because react-confetti no longer exports a named Props type. Inferring the props from the imported component keeps this wrapper aligned with the installed package type definitions.

Verification:

  • Inspected the reported Next.js build error and confirmed it points to the removed react-confetti Props import.
  • Ran git diff --check.
  • Not run locally: dependency installation could not complete because npm registry access failed from my environment while trying to install with a pnpm version compatible with this lockfile.

@vercel

vercel Bot commented May 26, 2026

Copy link
Copy Markdown

@alucard210319 is attempting to deploy a commit to the kxxoling's projects Team on Vercel.

A member of the Team first needs to authorize it.

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.

build failed

1 participant