-
Notifications
You must be signed in to change notification settings - Fork 362
[Docs] Adding ask AI Button #2858
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
base: trunk
Are you sure you want to change the base?
Conversation
- Introduced a new alias for `@docusaurus/ComponentCreator` in the Docusaurus configuration, pointing to a custom implementation in `src/patches/ComponentCreator.js`. - Implemented a dynamic loading mechanism for components using `react-loadable`, enhancing the routing and loading experience in the documentation site. - Added error handling for module exports to ensure robust loading of components.
@fellyph, what problem is this solving? Maintaining a patched version of a Docusaurus implementation detail seems like it would make our dependency on Docusaurus more fragile. It would be good to have more context around why this is needed. |
|
@brandonpayton, the changes were made to fix the issue with the pages that were crashing https://wordpress.github.io/wordpress-playground/api/blueprints/interface/EnableMultisiteStep/#Index And add back the Ask AI button, that was removed previously |
@fellyph ah, when I read
I thought that fixing the crash was a bonus on top of the other changes in this PR. I was about to ask more about patching the ComponentCreator, but it looks like that has been removed now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the changes were made to fix the issue with the pages that were crashing https://wordpress.github.io/wordpress-playground/api/blueprints/interface/EnableMultisiteStep/#Index
Besides the build plugin that references ComponentCreator, I'm not sure what changes remain that should fix the enable-multi-site step. It looks like maybe just adding the Kapa AI button remains. Am I missing something?
Updated comments for clarity and corrected casing in metadata description.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have not tested, but this looks good to me. Thanks!
This pull request introduces the Kapa AI integration into the Docusaurus documentation site. The main changes include adding a Kapa AI button to the navbar by swizzling the Search component and registering a new plugin for Kapa AI. Additionally, there are minor improvements to comments for clarity and consistency.
Kapa AI Integration:
swizzling) theSearchcomponent inNavbar/Search/index.js. The button triggers the Kapa AI widget when clicked.'./plugins/kapa-ai-plugin.js'to the plugins array indocusaurus.config.js.docusaurus.config.js.