fix(roundup): split embed fields by type per library#110
Merged
Conversation
Each library now gets a separator header field, then separate fields for Movies and Series/Episodes. Long sections spill into continuation fields instead of being silently truncated. Removes the old field_more key in favour of field_continued.
…ettings Replaces the single EMBED_SHOW_OVERVIEW flag with two independent options: EMBED_SHOW_OVERVIEW_MOVIES (movies + series) and EMBED_SHOW_OVERVIEW_EPISODES. Allows users to disable episode summaries to avoid spoilers without affecting movie/series notifications. Both default to true. Dashboard updated with two separate checkboxes; i18n keys added for en/de/sv.
Versions 1.0.0–1.15.1 had multiple high-severity issues including prototype pollution gadgets, SSRF bypass, CRLF injection, and credential injection. 1.16.0 addresses all reported advisories.
writeConfig now writes to a .tmp file and renames atomically, so a process kill during a write leaves the original config.json intact instead of corrupted/empty. updateConfig now refuses to write if readConfig() returns null, preventing a partial one-key file from replacing a fully configured config when secrets are auto-generated after a corrupted read.
…ge audit - api/jellyfin.js: log warn instead of silently swallowing errors in findLibraryByAncestors recursive search - utils/configFile.js: distinguish corrupt/unreadable config from "not found" in error message - bot/weeklyRoundup.js: throw on null getLibraryChannels(); show library-names-unavailable footer for single-library setups too; strip newlines from escapeMd; renderMovie/Series/Season now gracefully degrade to plain bold when itemDeeplink returns empty; remove dead isTest branch from onError - bot/roundupState.js: use type-safe check in migration guard instead of != null
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.
Changes
fix: Roundup embed fields split by content type
Embed fields per library showed all content in one block regardless of type. When a library had both movies and series, they were mixed together and long sections were silently truncated with an "… and N more" message instead of continuing in a second field.
field_morei18n key; addssection_movies,section_series,field_continuedin EN / DE / SV / template.feat: Separate overview toggle for movies and episodes
The embed overview setting is now split into two independent options -- one for movies and series, one for episodes. Allows users to disable episode summaries to avoid spoilers without affecting movie/series notifications.
EMBED_SHOW_OVERVIEWreplaced byEMBED_SHOW_OVERVIEW_MOVIESandEMBED_SHOW_OVERVIEW_EPISODES, both default on.