fix: windows installation tab on the docs#28125
fix: windows installation tab on the docs#28125Zamiell wants to merge 3 commits intooven-sh:mainfrom
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughAdds a Windows-specific tab selector UI component and a small client script. The component is imported into Changes
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 📝 Coding Plan
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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/installation.mdx`:
- Around line 6-8: The import references a missing React component
WindowsTabSelector; create the component file exports used by the docs or update
the import path. Add a new file docs/snippets/windows-tab-selector.jsx that
exports a default WindowsTabSelector React component (function or arrow) which
renders the expected tab UI (props if used by the docs) and export default
WindowsTabSelector, or if the component already exists elsewhere, change the
import in docs/installation.mdx to point to its actual location and
named/default export (WindowsTabSelector) to match the implementation.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: a2653ae4-f81b-4b9f-a09f-58e575f811a0
📒 Files selected for processing (2)
docs/installation.mdxdocs/os-tab-selector.js
What does this PR do?
Fixes #25042.
How did you verify your code works?
I ran
mintlify devand watched the Windows tab get auto selected.