Feature/add force recheck watchlist #5270
Open
Conversation
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
Add a backend endpoint and UI control to clear stale watchlist errors
and retrigger the Plex import job on demand. Update the Angular watchlist
view to show progress while the recheck runs, and cover the new service
logic with unit tests to verify we only delete existing errors when
needed.
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
There was a problem hiding this comment.
Pull Request Overview
This PR adds an admin "Force recheck" action for Plex watchlist functionality that clears existing watchlist errors and triggers an immediate import. The feature addresses issues where users experience authentication errors syncing their Plex watchlists.
- Adds a new API endpoint to clear watchlist errors and trigger job execution
- Implements a frontend button with loading state to trigger the recheck action
- Includes comprehensive unit tests for the new service functionality
Reviewed Changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| PlexController.cs | Adds new admin endpoint for revalidating watchlist users |
| plex-watchlist.component.ts | Implements force recheck functionality with loading state |
| plex-watchlist.component.html | Adds UI button and progress bar for force recheck |
| plex.service.ts | Adds HTTP service method for revalidation API call |
| PlexService.cs | Implements core logic to clear watchlist errors |
| IPlexService.cs | Adds interface method for force revalidation |
| PlexServiceTests.cs | Adds unit tests for the new revalidation functionality |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
src/Ombi/ClientApp/src/app/settings/plex/components/watchlist/plex-watchlist.component.ts
Show resolved
Hide resolved
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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
Added an admin “Force recheck” action that clears Plex watchlist errors, triggers an immediate import, and tests the new service logic.
🔗 Related Issues
#5246 (comment)
🧪 Testing
📸 Screenshots (if applicable)
📋 Checklist
🎯 Type of Change
📚 Additional Notes
This change was AI assisted but reviewed and validated by me.