Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TV Shows: Document "Specials in Seasons" feature #1107

Merged
merged 1 commit into from
Dec 12, 2024

Conversation

mihawk90
Copy link
Contributor

Noticed this wasn't documented anywhere, so added it.

Copy link
Member

@felix920506 felix920506 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add what the settings do? It's not really obvious from the setting names where the specials will play.

@mihawk90
Copy link
Contributor Author

mihawk90 commented Sep 10, 2024

It's not really obvious from the setting names where the specials will play.

Really? I feel like the names are fine, but IMO the order of fields in the Metadata editor is not ideal. "Airs before Episode" (field on the right) can be used in conjunction with "Airs before Season" (field on the left) to insert it somewhere in between. But (at least logically) it can't be used with "Airs after Season" (field in the middle) because.. well, presumably it's always at the end of the season where an "airs before episode" doesn't make sense.

I'll try writing something up.

edit:
Also while I'm at it... maybe someone knows for sure. If there are multiple Specials set as "airs before episode" 3 for example, I assume they are just sorted by air date? Or are they sorted by the S00EXX episode numbers? Does anyone know? I've only done this with the 3 Archer Specials above where both match up anyway.

I also just noticed you can actually fill all 3 fields, but "after Season" trumps the other 2 regardless of what was first set. Maybe not an ideal workflow (although one could argue that's a user error for inputting nonsensical data, but still).

@mihawk90 mihawk90 force-pushed the tvshows-specials-in-seasons branch from 67be299 to 7b5528c Compare September 11, 2024 00:07
@mihawk90
Copy link
Contributor Author

Updated according to the above comments.

Should be noted that I documented it how it should work, but there is currently jellyfin/jellyfin-web#6049 where the Season's Play button doesn't behave correctly.
Should that be added as a note?

@felix920506
Copy link
Member

is this ready for review?

@mihawk90
Copy link
Contributor Author

Sure, I've nothing else to add. The mentioned issue also has a pending PR for a fix so that's not needed anymore either.

docs/general/server/media/shows.md Outdated Show resolved Hide resolved
docs/general/server/media/shows.md Outdated Show resolved Hide resolved
docs/general/server/media/shows.md Outdated Show resolved Hide resolved
docs/general/server/media/shows.md Outdated Show resolved Hide resolved
docs/general/server/media/shows.md Outdated Show resolved Hide resolved
docs/general/server/media/shows.md Outdated Show resolved Hide resolved
@mihawk90 mihawk90 force-pushed the tvshows-specials-in-seasons branch from 7b5528c to 50fad4c Compare December 12, 2024 13:57
@mihawk90
Copy link
Contributor Author

All changed and also rebased onto current master

@jellyfin-bot
Copy link

Cloudflare Pages deployment

Latest commit 50fad4c0633c9ba01395cf2ef465a3beac2dfe2f
Status ✅ Deployed!
Preview URL https://ffe2c3b6.jellyfin-org.pages.dev
Type 🔀 Preview

@Shadowghost Shadowghost merged commit 1434bc2 into jellyfin:master Dec 12, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants