-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
feat: add initial RSC support #6646
Conversation
Thanks for doing the initial work. These restrictions from react feel so arbitrary...I thought the point of "use client" was that the components would just work, it's strange that there are these additional bonus things. Do we know if they are RSC or next restrictions and are they temporary/short term maybe? I guess the work arounds aren't a too bad but it's pretty annoying to have to copy paste a bunch of factories as well as export everything |
The first one looks like a limitation of RSC whereas the second issue looks like a bug. I might just publish a beta of RB so I can post an issue with repro into the nextjs github board. I'd like some definitive answers since I couldn't find much in my searches. |
Closes #6475
Enables the plugin to add
use client
into all of the components.Found some issues though:
1. Cannot use a component that has been created with
createWithBsPrefix
. For example:CardGroup
Potential solution:
We would need to stop using this function to create components.
2. Cannot use a component that's been set as a property. For example
Accordion.Body
Potential solution:
Users would have to import the component separately
import AccordionBody from 'react-bootstrap/AccordionBody'