OPNsense checkbox "Show community plugins"#765
Conversation
OPNsense requires a new checkbox "Show community plugins" to be checked, before you can find the netbird plugin. Tested on OPNsense 26.1.8_5
📝 WalkthroughWalkthroughUpdated OPNsense NetBird plugin installation documentation to explicitly instruct users to enable "Show community plugins" in the Plugins page before searching for and installing the ChangesOPNsense Plugin Installation Guide
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
src/pages/get-started/install/opnsense.mdxOops! Something went wrong! :( ESLint: 9.39.4 TypeError: Converting circular structure to JSON 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
src/pages/get-started/install/opnsense.mdx (1)
1-1:⚠️ Potential issue | 🟠 Major | ⚡ Quick winAdd missing imports for custom components.
The file uses custom components (
YouTube,Note,Button) without importing them. As per coding guidelines, MDX files must import these components from their specified paths.📦 Proposed fix to add imports
+import { Note } from '`@/components/mdx`' +import { YouTube } from '`@/components/YouTube`' +import { Button } from '`@/components/Button`' + # OPNsense InstallationAs per coding guidelines: Import custom components from
@/components/mdx(Note, Warning, Success),@/components/Tiles,@/components/YouTube,@/components/Button, or@/components/mdxas needed for alerts, layout, media, and UI🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/get-started/install/opnsense.mdx` at line 1, This MDX uses custom components YouTube, Note, and Button but lacks imports; add import statements at the top of src/pages/get-started/install/opnsense.mdx to import YouTube, Note, and Button from the project's MDX/component modules (the canonical component exports used across MDX files), then update any existing usages to rely on those imports (symbols: YouTube, Note, Button) so the components render correctly.
🧹 Nitpick comments (1)
src/pages/get-started/install/opnsense.mdx (1)
26-26: ⚡ Quick winFix comma splice and improve sentence structure.
The instruction contains a comma splice: "...
Plugins, Make sure..." should be separated into distinct sentences or use proper punctuation for better readability.✍️ Proposed fix for grammar and clarity
- In the OPNsense Web UI, navigate to `System` > `Firmware` > `Plugins`, Make sure `Show community plugins` is checked and search for the `os-netbird` package. Click the install button next to it. + In the OPNsense Web UI, navigate to `System` > `Firmware` > `Plugins`. Make sure `Show community plugins` is checked, then search for the `os-netbird` package and click the install button next to it.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/get-started/install/opnsense.mdx` at line 26, Split the comma splice by replacing the comma after the backticked `Plugins` with a period and start a new sentence (or use a semicolon and lowercase the following word); update the fragment "`System` > `Firmware` > `Plugins`, Make sure `Show community plugins` is checked..." to read either "`System` > `Firmware` > `Plugins`. Make sure `Show community plugins` is checked..." or "`System` > `Firmware` > `Plugins`; make sure `Show community plugins` is checked..." and keep the rest of the instruction referencing the `os-netbird` package unchanged.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Outside diff comments:
In `@src/pages/get-started/install/opnsense.mdx`:
- Line 1: This MDX uses custom components YouTube, Note, and Button but lacks
imports; add import statements at the top of
src/pages/get-started/install/opnsense.mdx to import YouTube, Note, and Button
from the project's MDX/component modules (the canonical component exports used
across MDX files), then update any existing usages to rely on those imports
(symbols: YouTube, Note, Button) so the components render correctly.
---
Nitpick comments:
In `@src/pages/get-started/install/opnsense.mdx`:
- Line 26: Split the comma splice by replacing the comma after the backticked
`Plugins` with a period and start a new sentence (or use a semicolon and
lowercase the following word); update the fragment "`System` > `Firmware` >
`Plugins`, Make sure `Show community plugins` is checked..." to read either
"`System` > `Firmware` > `Plugins`. Make sure `Show community plugins` is
checked..." or "`System` > `Firmware` > `Plugins`; make sure `Show community
plugins` is checked..." and keep the rest of the instruction referencing the
`os-netbird` package unchanged.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 2959613a-f897-438d-89b2-a598a06991ac
📒 Files selected for processing (1)
src/pages/get-started/install/opnsense.mdx
1 participant
OPNsense requires a new checkbox "Show community plugins" to be checked, before you can find the netbird plugin.
Tested on OPNsense 26.1.8_5
Summary by CodeRabbit