Skip to content

Fix: Corrected tutorial code for CardWrapper component #969

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Fix: Corrected tutorial code for CardWrapper component #969

wants to merge 1 commit into from

Conversation

tolvap
Copy link

@tolvap tolvap commented Jan 5, 2025

Attached is an image of what is being rendered after replacing Card components with Suspense and CardWrapper component in chapter 9. The issue is that after deleting the initial implementation of Card component, the CardWrapper was initialized with a React Fragment instead of a styled container div.
image

Copy link

vercel bot commented Jan 5, 2025

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 Jan 5, 2025 9:06pm
next-seo-starter ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 5, 2025 9:06pm

Copy link

vercel bot commented Jan 5, 2025

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

A member of the Team first needs to authorize it.

@tolvap
Copy link
Author

tolvap commented Jan 5, 2025

Found out that this container was included in the resulting code, just outside of a Suspense. Closing the PR.

@tolvap tolvap closed this Jan 5, 2025
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.

1 participant