overseerr icon indicating copy to clipboard operation
overseerr copied to clipboard

feat: add support for requesting "Specials" for TV Shows

Open AhmedNSidd opened this issue 1 year ago • 8 comments

Description

This pull request is responsible for adding support in Overseerr for requesting "Special" episodes for TV Shows. This request has become especially pertinent when you consider shows like "Doctor Who". These shows have Specials that are critical to understanding the plot of a TV show.

Screenshot (if UI-related)

The TV Show Page

If you navigate to any show with Specials, you'll see the "Specials" season just before "Season 1". NOTE: In this example, Doctor Who has 13 Seasons + Specials. However, in the header for the TV show, we STILL say that the show has 13 seasons (we don't include specials as a "Season" in the headers).

CleanShot 2023-12-24 at 14 56 10@2x
The TV Show Page (Request Modal)

NOTE: Instead of having the table say Season 0 which it would normally say for the Specials season, it says "Specials". Whether it should say Season 0 or Specials is a discussion that could be worth having for this PR, but I personally went with Specials for more user-friendliness.

CleanShot 2023-12-24 at 15 01 51@2x
The TV Show Page (Manage Series Sidebar)

NOTE: Instead of having the Badge say 0 which it would normally say for the Specials season, it says "Specials". Whether it should say 0 or Specials is a discussion that could be worth having for this PR, but I personally went with Specials for more user-friendliness.

CleanShot 2023-12-24 at 14 59 36@2x
The Requests Page
CleanShot 2023-12-24 at 15 02 50@2x

To-Dos

  • [x] Successful build yarn build
  • [X] Translation keys yarn i18n:extract
  • [ ] Database migration (if required)

Issues Fixed or Closed

  • Fixes #779

AhmedNSidd avatar Dec 24 '23 23:12 AhmedNSidd

Can we remove the yarn.lock from this PR? It shouldn't be getting updated here.

sct avatar Jan 10 '24 02:01 sct

Any updates on this?

Panda260 avatar Mar 17 '24 10:03 Panda260

@Panda260 No new updates.. I've updated the branch to be rebased on top of the develop branch. I'm not sure what the next steps are for having this merged in. I believe I need an approval from @sct / @TheCatLady / @danshilm / @OwsleyJr

AhmedNSidd avatar Apr 02 '24 00:04 AhmedNSidd