Skip to content
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

Fix component rendering for .ts components #60

Merged
merged 3 commits into from
Nov 25, 2024

Conversation

damien-schneider
Copy link
Owner

This pull request includes several changes across multiple files to update the import and usage of SectionsList to sectionList from the @cuicui/ui/lib/section-list module. Additionally, there are some minor improvements and refactorings in component files. The most important changes are summarized below:

Updates to sectionList imports and usage:

Minor improvements and refactorings:

  • Changed the component prop handling in single-component-section.tsx to remove unnecessary object spread. (apps/website/src/app/(site)/[section]/[category]/single-component-section.tsxL30-R30)
  • Updated the MainMenuCardContent component to remove the category prop from the createElement call.
  • Updated the ComponentTabRenderer to use isValidElement for rendering components. [1] [2] [3]
  • Added missing fields (name, description) to the macOsDropdownComponent object in component.ts. [1] [2]
  • Renamed and updated the BarCodeGeneratorComponent in component.ts to use named exports for PreviewBarCodeGenerator. [1] [2]

Copy link

vercel bot commented Nov 25, 2024

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

Name Status Preview Comments Updated (UTC)
cuicui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 25, 2024 2:01pm

@damien-schneider damien-schneider merged commit f5c8fcb into main Nov 25, 2024
3 checks passed
@damien-schneider damien-schneider deleted the fix-component-rendering-for-.ts-components branch November 25, 2024 14:05
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