fix: honor Jellyfin played threshold#2466
Conversation
|
Maybe related? |
|
I tested this out and the code changes look good too. Nice way to use manually marked items in a |
|
I did merge this into the main branch, but it is worth noting that the Plex side has an issue. Well, two issues. The data that is being used in that change is cached for 5 minutes. Would not be/could not be a problem unless you are trying to run rules or test media shortly after marking something as watched. The fix is to use the PlexMetadata pulled at the top of the getter, which is already called and has per item cache updates. It currently does not have viewCount as a returned item. Adding it isn't a solution either as that call, and its results, are per the user token who pulled the metadata (whoever is authenticated in Maintainerr and usually admin). So viewCount for the admin user might be 1 and another user looking at viewCount would see 2 (only there two watches). Maintainerr only sees I do not have multiple Jellyfin users so I couldn't test this on their side. |
…, #2442, #2406, #2386, #2370 PR #2466 - fix: honor Jellyfin played threshold - Respect configured played percentage threshold for Jellyfin watch status PR #2461 - feat(rules): add ARR disk target path selection for disk space rules - Allow selecting specific disk target paths for Radarr/Sonarr disk space rules PR #2458 - feat: clean up empty ended shows in Sonarr after season actions - Automatically remove ended shows from Sonarr when all seasons are processed PR #2453 - fix: improve Plex viewCount reliability and add isWatched boolean - Use native Plex viewCount field with watch history fallback - Add new isWatched boolean rule property PR #2452 - build(deps): bump actions/download-artifact from 7 to 8 PR #2451 - build(deps): bump actions/upload-artifact from 6 to 7 PR #2442 - fix(server): reject null/undefined in numeric rule comparisons - Add getComparisonResult wrapper that fails closed on null/undefined operands - Strict type checking for BIGGER/SMALLER comparisons PR #2406 - Metadata provider abstraction layer with TVDB support - Add MetadataService as central metadata resolution layer - TVDB support as alternative metadata provider - Dynamic provider preference with fallback - Replace TmdbIdService with unified MetadataService PR #2386 - feat: missing_episode rules - Add missing episode count as a rule property for Sonarr PR #2370 - build(deps-dev): bump the eslint group with 2 updates
|
This is now included in the You can check the latest commits here: https://github.com/Maintainerr/Maintainerr/commits/jellyfin-dev
Thank you very much! 🚀 |
|
🎉 This PR is included in version 3.1.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
What:
Use Jellyfin's own played threshold when building watch history instead of relying only on the Played flag.
Why:
This makes the Jellyfin "Viewed by" behavior match Jellyfin's configured watched semantics.
Fixes #2465
Note:
Not fully tested. This is a POC.