Tell users how to fix authors skipped by bulk sync-monitoring - #162
Open
jordanfelle wants to merge 2 commits into
Open
jordanfelle wants to merge 2 commits into
jordanfelle wants to merge 2 commits into
Conversation
The bulk Author Editor's Sync Monitored Audio/eBooks action silently skips any author missing one of the two root folder paths, reporting only an aggregate count with no indication of what to do next. Add the fix (set both root folder paths, then retry) to the warning text. Fixes Chaptarr#161
The previous fix (9725c25) told users to "set both root folder paths" without saying how - and it's not obvious: root folder is a per-media-type bulk field (Author Editor already supports setting it, same as quality/metadata profile), gated behind the media-type filter tab, not a general field. You have to switch the filter to Audiobooks, bulk-set root folder for the skipped authors, switch to eBooks, repeat, then re-run sync. That capability already exists in Author Editor (RootFolderSelectInputConnector wired up in AuthorEditorFooter.js) - this was mistakenly reported as missing from the frontend in earlier investigation. No frontend change needed; just point the warning at the real steps instead of vague/incorrect guidance.
jordanfelle
added a commit
to jordanfelle/chaptarr
that referenced
this pull request
Sep 26, 2026
jordanfelle
added a commit
to jordanfelle/chaptarr
that referenced
this pull request
Sep 26, 2026
jordanfelle
added a commit
to jordanfelle/chaptarr
that referenced
this pull request
Sep 26, 2026
jordanfelle
added a commit
to jordanfelle/chaptarr
that referenced
this pull request
Sep 27, 2026
jordanfelle
added a commit
to jordanfelle/chaptarr
that referenced
this pull request
Sep 27, 2026
jordanfelle
added a commit
to jordanfelle/chaptarr
that referenced
this pull request
Sep 27, 2026
jordanfelle
added a commit
to jordanfelle/chaptarr
that referenced
this pull request
Sep 27, 2026
jordanfelle
added a commit
to jordanfelle/chaptarr
that referenced
this pull request
Sep 27, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Fixes #161. The bulk Author Editor's "Sync Monitored Audio/eBooks" action silently skips any author missing one of the two root folder paths (
AudiobookRootFolderPath/EbookRootFolderPath), reporting only an aggregate count with no indication of what to do next.Correction from the original version of this PR: I initially assumed the frontend bulk editor had no way to set root folder paths at all, and considered adding a new UI control for it. That assumption was wrong - Author Editor already has a Root Folder bulk-set field (
AuthorEditorFooter.js, backed byRootFolderSelectInputConnector), the same way it already supports bulk quality/metadata profile. It's just gated behind the page's media-type filter tab (Audiobooks / eBooks / All), same as quality and metadata profile are, so it only applies to one media type at a time - you can't set both root folders in a single action. That's the actual reason people hit the "both root folders required" skip: they set one via the bulk editor and assume that's enough, or don't realize the fix already exists in the UI.So no frontend change is needed here - the fix is just telling people the real, already-supported steps instead of vague or (in the previous revision of this PR) actively wrong guidance:
Database Migration
NO
How was this tested?
Updated the existing unit test in
AuthorEditorMissingRootHydrationFixture.csto assert on the new warning text.dotnet test --filter FullyQualifiedName~AuthorEditorMissingRootHydrationFixture- 5/5 passing.Did not test the frontend live (no running instance in this sandbox) - no frontend files were touched, so this is a backend-only string change plus a matching test update.
Screenshots (UI changes only)
N/A - backend response header text only.