docs(truenas): update install/migration guide#2491
Conversation
|
No actionable comments were generated in the recent review. 🎉 📝 WalkthroughWalkthroughRemoved WIP/deprecation warnings from TrueNAS docs and added a concise Install section; expanded migration-guide with tabbed Host Path and ixVolume migration flows and updated Unraid migration steps; moved Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (2)
docs/migration-guide.mdx (1)
240-243: The rsync source path assumes Jellyseerr; Overseerr users may need a different path.The rsync command hardcodes
/mnt/.ix-apps/app_mounts/jellyseerr/as the source. Users migrating from Overseerr may have their data at a different path (e.g.,/mnt/.ix-apps/app_mounts/overseerr/). Consider adding a note to adjust the path accordingly.📝 Suggested clarification
3. Copy ixVolume Data Open System Settings → Shell, or SSH into your TrueNAS server as root and run : ``` - rsync -av /mnt/.ix-apps/app_mounts/jellyseerr/ /mnt/storage/seerr/ + rsync -av /mnt/.ix-apps/app_mounts/<app-name>/ /mnt/storage/seerr/ ``` + Replace `<app-name>` with `jellyseerr` or `overseerr` depending on your current installation.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/migration-guide.mdx` around lines 240 - 243, The docs currently hardcode the rsync source as /mnt/.ix-apps/app_mounts/jellyseerr/ which is incorrect for Overseerr users; update the migration example in migration-guide.mdx to use a placeholder (e.g., /mnt/.ix-apps/app_mounts/<app-name>/) and add a brief note instructing readers to replace <app-name> with jellyseerr or overseerr (or their actual app folder) so the rsync command works for both installations.docs/getting-started/third-parties/truenas.mdx (1)
19-22: Install section provides guidance despite WIP warning.The danger block at lines 7-10 states this method "has not yet been updated for Seerr and is currently a work in progress," yet this new Install section provides concrete instructions. Consider either:
- Updating the danger block to reflect that basic installation works but may have issues, or
- Adding a note that these are preliminary instructions pending the linked issue resolution.
This will help users understand what to expect.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/getting-started/third-parties/truenas.mdx` around lines 19 - 22, The new "## Install" section gives concrete steps for installing Seerr but contradicts the earlier danger block that marks the method as WIP; update the danger block near the top of the file or add a short inline note inside the "## Install" section to clarify these are preliminary instructions (e.g., "preliminary — may have issues; see linked issue") so readers know the install steps (Apps menu → Discover Apps → search 'Seerr') are provisional until the linked issue is resolved; modify either the existing danger block text or insert a brief caveat at the start of the "## Install" section to resolve the inconsistency.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@docs/getting-started/third-parties/truenas.mdx`:
- Around line 19-22: The new "## Install" section gives concrete steps for
installing Seerr but contradicts the earlier danger block that marks the method
as WIP; update the danger block near the top of the file or add a short inline
note inside the "## Install" section to clarify these are preliminary
instructions (e.g., "preliminary — may have issues; see linked issue") so
readers know the install steps (Apps menu → Discover Apps → search 'Seerr') are
provisional until the linked issue is resolved; modify either the existing
danger block text or insert a brief caveat at the start of the "## Install"
section to resolve the inconsistency.
In `@docs/migration-guide.mdx`:
- Around line 240-243: The docs currently hardcode the rsync source as
/mnt/.ix-apps/app_mounts/jellyseerr/ which is incorrect for Overseerr users;
update the migration example in migration-guide.mdx to use a placeholder (e.g.,
/mnt/.ix-apps/app_mounts/<app-name>/) and add a brief note instructing readers
to replace <app-name> with jellyseerr or overseerr (or their actual app folder)
so the rsync command works for both installations.
e447c84 to
1502e65
Compare
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/migration-guide.mdx`:
- Around line 213-248: The TabItem values/labels are swapped relative to their
content: the TabItem with value="ixvolume" currently contains Host Path
migration steps and the TabItem with value="hostpath" contains ixVolume steps;
update the two TabItem elements (the ones with value="ixvolume" and
value="hostpath") so their value and label match the described content (swap the
labels and/or values so the "Host Path" label/value wraps the Host Path
migration steps and the "ixVolume" label/value wraps the ixVolume migration
steps).
Signed-off-by: Ludovic Ortega <ludovic.ortega@adminafk.fr>
1502e65 to
0938d23
Compare
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/migration-guide.mdx`:
- Around line 223-245: The rsync example hardcodes "jellyseerr" and copies to
/mnt/storage/seerr/ which conflicts with Step 4's host path
/mnt/storage/seerr/config; update the rsync line (the block under "3. Copy
ixVolume Data") to use a placeholder like /mnt/.ix-apps/app_mounts/<app-name>/
and make the destination include the config subfolder
(/mnt/storage/<app-name>/config/) or alternatively change the Step 4 host path
to match the current rsync destination; also add a short note: "Replace
<app-name> with jellyseerr or overseerr as appropriate." Reference the rsync
command and Step 4 host path when making edits.
|
Yes, I fixed it after CoderabbitAI reported it to me, but I didn’t refresh the screenshots. |
There was a problem hiding this comment.
Pull request overview
This pull request updates the TrueNAS installation and migration documentation based on community feedback from Discord users. The changes remove the work-in-progress warnings and add concrete installation and migration instructions for TrueNAS users. The PR also attempts to address a deprecated Docusaurus configuration warning.
Changes:
- Added installation instructions for TrueNAS in the third-party documentation
- Created comprehensive migration guide for TrueNAS with two migration paths (Host Path and ixVolume)
- Attempted to fix deprecated
onBrokenMarkdownLinksconfiguration in Docusaurus
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 6 comments.
| File | Description |
|---|---|
| gen-docs/docusaurus.config.ts | Attempts to move deprecated onBrokenMarkdownLinks config to new location (contains error) |
| docs/migration-guide.mdx | Adds detailed TrueNAS migration instructions with Host Path and ixVolume migration paths |
| docs/getting-started/third-parties/truenas.mdx | Removes work-in-progress warning and adds basic installation instructions |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This PR contains the following updates: | Package | Update | Change | |---|---|---| | [ghcr.io/seerr-team/seerr](https://github.com/seerr-team/seerr) | minor | `v3.0.1` → `v3.1.0` | | [seerr-team/seerr](https://github.com/seerr-team/seerr) | minor | `v3.0.1` → `v3.1.0` | --- ### Release Notes <details> <summary>seerr-team/seerr (ghcr.io/seerr-team/seerr)</summary> ### [`v3.1.0`](https://github.com/seerr-team/seerr/releases/tag/v3.1.0) [Compare Source](seerr-team/seerr@v3.0.1...v3.1.0) ##### 🛡️ Security - Patch [CVE-2026-27707](GHSA-rc4w-7m3r-c2f7) - Unauthenticated account registration on Plex-configured Seerr instances via Jellyfin authentication endpoint - ([4ae2068](seerr-team/seerr@4ae2068)) - Patch [CVE-2026-27793](GHSA-f7xw-jcqr-57hp) - Broken Object-Level Authorization in User Profile Endpoint Exposes Third-Party Notification Credentials - ([4f089b2](seerr-team/seerr@4f089b2)) - Patch [CVE-2026-27792](GHSA-gx3h-3jg5-q65f) - Missing authentication on pushSubscription endpoints - ([946bdecec](seerr-team/seerr@946bdec)) ##### 🚀 Features - *(helm)* Use an existing PVC as config volume ([#​2447](seerr-team/seerr#2447)) - ([8f0c904](seerr-team/seerr@8f0c904)) - *(servarr-api)* Make Servarr API request timeout configurable ([#​2556](seerr-team/seerr#2556)) - ([3bcb4da](seerr-team/seerr@3bcb4da)) - Self-host font for better privacy ([#​2540](seerr-team/seerr#2540)) - ([10ea21b](seerr-team/seerr@10ea21b)) ##### 🐛 Bug Fixes - *(helm)* Add "v" as prefix for appVersion tag ([#​2445](seerr-team/seerr#2445)) - ([04b9d87](seerr-team/seerr@04b9d87)) - *(jellyfin-scanner)* Include unmatched seasons in processable seasons ([#​2538](seerr-team/seerr#2538)) - ([68f56d2](seerr-team/seerr@68f56d2)) - *(link-account)* Fix error-message override ([#​2547](seerr-team/seerr#2547)) - ([b843be0](seerr-team/seerr@b843be0)) - *(plex-scanner)* Add TVDb to TMDB fallback in plex scanner ([#​2537](seerr-team/seerr#2537)) - ([7c60a5c](seerr-team/seerr@7c60a5c)) - *(radarr)* Trigger search for existing monitored movies without files ([#​2391](seerr-team/seerr#2391)) - ([55776ea](seerr-team/seerr@55776ea)) - *(servarr)* Increase default API timeout from 5000ms to 10000ms ([#​2442](seerr-team/seerr#2442)) - ([b499976](seerr-team/seerr@b499976)) - *(sonarr)* Use configured metadata provider for season filtering ([#​2516](seerr-team/seerr#2516)) - ([5013d1d](seerr-team/seerr@5013d1d)) - *(watch-data)* Use sentinel values to avoid invalid SQL syntax ([#​2552](seerr-team/seerr#2552)) - ([947f70c](seerr-team/seerr@947f70c)) - *(watchlist-sync)* Correct permission typo for TV auto requests ([#​2488](seerr-team/seerr#2488)) - ([e0e4b6f](seerr-team/seerr@e0e4b6f)) - Preserve blocklist on media deletion & optimise watchlist-sync ([#​2478](seerr-team/seerr#2478)) - ([9da8bb6](seerr-team/seerr@9da8bb6)) ##### 🚜 Refactor - *(tailwind)* Replace deprecated tailwind utilities ([#​2542](seerr-team/seerr#2542)) - ([f42a4ec](seerr-team/seerr@f42a4ec)) ##### 📖 Documentation - *(synology)* Add installation guide via SynoCommunity ([#​2503](seerr-team/seerr#2503)) - ([0e636a3](seerr-team/seerr@0e636a3)) - *(truenas)* Update install/migration guide ([#​2491](seerr-team/seerr#2491)) - ([dc1734d](seerr-team/seerr@dc1734d)) - *(unraid)* Improve unraid migration guide ([#​2470](seerr-team/seerr#2470)) - ([5e64d49](seerr-team/seerr@5e64d49)) - Update Unraid install and migration guides with dual permission methods ([#​2532](seerr-team/seerr#2532)) - ([a0d0eb1](seerr-team/seerr@a0d0eb1)) - Add a warning in migration-guide for third party installation ([#​2527](seerr-team/seerr#2527)) - ([7e9dff3](seerr-team/seerr@7e9dff3)) - Remove double quotes (") from DB\_HOST environment variable ([#​2514](seerr-team/seerr#2514)) - ([fa905be](seerr-team/seerr@fa905be)) - Add Unraid installation and migration guide ([#​2440](seerr-team/seerr#2440)) - ([b6a9132](seerr-team/seerr@b6a9132)) - Fix migration guide title ([#​2425](seerr-team/seerr#2425)) - ([39ae32f](seerr-team/seerr@39ae32f)) ##### ⚡ Performance - Add missing indexes on all foreign key columns ([#​2461](seerr-team/seerr#2461)) - ([c6bcfe0](seerr-team/seerr@c6bcfe0)) ##### ⚙️ Miscellaneous Tasks - *(changelog)* Fix changelog template ([#​2431](seerr-team/seerr#2431)) - ([c2977f6](seerr-team/seerr@c2977f6)) - *(eslint)* Add react/self-closing-comp ([#​2563](seerr-team/seerr#2563)) - ([cd8b386](seerr-team/seerr@cd8b386)) - *(github)* Add docs and maintenance issue templates ([#​2467](seerr-team/seerr#2467)) - ([cf4883a](seerr-team/seerr@cf4883a)) - *(helm)* Add GatewayAPI route support to helm chart ([#​2544](seerr-team/seerr#2544)) - ([3a42f59](seerr-team/seerr@3a42f59)) - *(helm)* Update ghcr.io/seerr-team/seerr ( 3.0.0 → 3.0.1 ) \[skip-ci] ([#​2441](seerr-team/seerr#2441)) - ([87fb0df](seerr-team/seerr@87fb0df)) - *(husky)* Fixed husky commit message from bash/zsh syntax to sh syntax ([#​2572](seerr-team/seerr#2572)) - ([a00c9e5](seerr-team/seerr@a00c9e5)) - *(release)* Prepare ${TAG\_VERSION} - ([94a70bb](seerr-team/seerr@94a70bb)) - Updated the Contributing and Security guides to reflect our current practices ([#​2579](seerr-team/seerr#2579)) - ([0d40a42](seerr-team/seerr@0d40a42)) - Disable nextjs telemetry ([#​2517](seerr-team/seerr#2517)) - ([cecdd63](seerr-team/seerr@cecdd63)) - Update contributing guide regarding Automated AI Agent ([#​2518](seerr-team/seerr#2518)) - ([880fbc9](seerr-team/seerr@880fbc9)) - Remove discord notification from release ([#​2501](seerr-team/seerr#2501)) - ([fba20c1](seerr-team/seerr@fba20c1)) - Add create-tag workflow to streamline release process ([#​2493](seerr-team/seerr#2493)) - ([06e5eb0](seerr-team/seerr@06e5eb0)) - Update concurrency logic ([#​2481](seerr-team/seerr#2481)) - ([4939f13](seerr-team/seerr@4939f13)) - Add semantic-pr workflow to enforce conventional commits ([#​2472](seerr-team/seerr#2472)) - ([5e57fdc](seerr-team/seerr@5e57fdc)) ##### New Contributors ❤️ - [@​caillou](https://github.com/caillou) made their first contribution - [@​Kenshin9977](https://github.com/Kenshin9977) made their first contribution - [@​MagicLegend](https://github.com/MagicLegend) made their first contribution - [@​wiiaam](https://github.com/wiiaam) made their first contribution - [@​mjonkus](https://github.com/mjonkus) made their first contribution - [@​nova-api](https://github.com/nova-api) made their first contribution - [@​mreid-tt](https://github.com/mreid-tt) made their first contribution - [@​DataBitz](https://github.com/DataBitz) made their first contribution - [@​Hyperion2220](https://github.com/Hyperion2220) made their first contribution - [@​blassley](https://github.com/blassley) made their first contribution - [@​JanKleine](https://github.com/JanKleine) made their first contribution - [@​koiralasandesh](https://github.com/koiralasandesh) made their first contribution<!-- generated by git-cliff --> </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about these updates again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0My4yNS43IiwidXBkYXRlZEluVmVyIjoiNDMuMjUuNyIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsiaW1hZ2UiXX0=--> Reviewed-on: https://gitea.alexlebens.dev/alexlebens/infrastructure/pulls/4284 Co-authored-by: Renovate Bot <renovate-bot@alexlebens.net> Co-committed-by: Renovate Bot <renovate-bot@alexlebens.net>
Signed-off-by: Ludovic Ortega <ludovic.ortega@adminafk.fr>

Description
docs(truenas): update install/migration guide
I haven’t tested the guide myself, but some Discord users have reported that it works.
based on : truenas/apps#3374 (comment)
based on : truenas/apps#3374 (comment)
Also fixed
How Has This Been Tested?
Screenshots / Logs (if applicable)
Host path

ixVolume

Checklist:
pnpm buildpnpm i18n:extractSummary by CodeRabbit