Conversation
…oves Node-20 deprecation warnings) [skip ci]
…oves Node-20 deprecation warnings) [skip ci]
…oves Node-20 deprecation warnings) [skip ci]
…oves Node-20 deprecation warnings) [skip ci]
ci: migrate CodeQL runner to self-hosted ARC (develop)
ci: migrate CodeQL runner to self-hosted ARC (stage)
Feat/docs UI revamp
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Not up to standards ⛔
|
Greptile SummaryRefreshes the documentation site’s design and release infrastructure.
Confidence Score: 5/5The PR appears safe to merge with no concrete changed-code failure identified. The documentation routes, Docusaurus theme extension points, conditional search configuration, and sidebar customizations remain consistent with the repository’s current configuration and dependencies.
What T-Rex did
|
| Filename | Overview |
|---|---|
| website/docusaurus.config.ts | Updates branding, navigation, search placement, fonts, footer links, and theme configuration without an identified actionable defect. |
| website/docs/advanced-guide/support.mdx | Replaces the placeholder support document with a structured support guide and reusable presentation components. |
| website/src/components/SidebarPromo/index.tsx | Adds a route-specific promotional panel for the desktop support-page sidebar. |
| website/src/theme/MDXComponents.tsx | Wraps Markdown tables in keyboard-focusable horizontal scrolling containers. |
| website/sidebars.ts | Adds Kit navigation and retains hidden Advanced Guide entries needed for document sidebar association. |
| .github/workflows/k8s-build.yml | Updates runner selection and Docker-related action major versions for the image publishing workflow. |
| .github/workflows/codeql.yml | Updates runner selection and the checkout action for CodeQL analysis. |
Comments Outside Diff (1)
-
General comment
Support MDX causes full-page React hydration failure
- Bug
- Opening the production
/advanced-guide/support/route produces 42 recoverable hydration errors (React error #418) followed by error #423, meaning React abandons the server-rendered tree and client-renders the root. The UI eventually appears, but users pay unnecessary rerendering cost and the browser console is flooded. The comparable/advanced-guide/route produces zero errors.
- Opening the production
- Cause
- The changed
website/docs/advanced-guide/support.mdxplaces blank-line Markdown content inside literal JSX<p>elements at lines 121, 132, 321, and 369. Generated HTML demonstrates that these become empty styled paragraphs followed by separate unstyled paragraphs—for example<p class="heroLead_QYIT"></p><p>Ever Teams is developed…</p>—creating server/client markup disagreement across many elements.
- The changed
- Fix
- Keep paragraph text directly inside each JSX
<p>without separating blank lines, or replace these wrappers with MDX-safe components that acceptchildren. Rebuild and verify that the production route reports zeroonRecoverableErrorconsole events in both themes.
- Keep paragraph text directly inside each JSX
- Bug
Reviews (1): Last reviewed commit: "Merge pull request #72 from ever-co/deve..." | Re-trigger Greptile
No description provided.