From 597f6ecf1644d5d6ace127d7d1e055a4ed3fd2c9 Mon Sep 17 00:00:00 2001 From: shamoon <4887959+shamoon@users.noreply.github.com> Date: Sun, 15 Feb 2026 18:44:03 -0800 Subject: [PATCH] Enhancement: jellyseer completed (#6329) --- docs/widgets/services/index.md | 2 +- docs/widgets/services/seerr.md | 10 +++++----- public/locales/en/common.json | 1 + src/widgets/seerr/component.jsx | 13 +++++++++++-- src/widgets/seerr/component.test.jsx | 24 +++++++++++++++++++++--- src/widgets/seerr/widget.test.js | 2 +- 6 files changed, 40 insertions(+), 12 deletions(-) diff --git a/docs/widgets/services/index.md b/docs/widgets/services/index.md index 4aa67bdd6..de8c4ce61 100644 --- a/docs/widgets/services/index.md +++ b/docs/widgets/services/index.md @@ -67,7 +67,7 @@ You can also find a list of all available service widgets in the sidebar navigat - [Jackett](jackett.md) - [JDownloader](jdownloader.md) - [Jellyfin](jellyfin.md) -- [Jellyseerr](jellyseerr.md) +- [Seerr](seerr.md) - [Jellystat](jellystat.md) - [Kavita](kavita.md) - [Komga](komga.md) diff --git a/docs/widgets/services/seerr.md b/docs/widgets/services/seerr.md index 828722337..09f460bbe 100644 --- a/docs/widgets/services/seerr.md +++ b/docs/widgets/services/seerr.md @@ -7,14 +7,14 @@ Learn more about [Seerr](https://github.com/seerr-team/seerr). Find your API key under `Settings > General > API Key`. -_Note that Jellyseerr was merged with Overseerr and renamed Seerr._ +_Note that Jellyseerr was merged with Overseerr and renamed Seerr. Use `type: seerr` (legacy `type: jellyseerr` is aliased)._ -Allowed fields: `["pending", "approved", "available", "issues"]`. -Default fields: `["pending", "approved", "available"]`. +Allowed fields: `["pending", "approved", "available", "completed", "issues"]`. +Default fields: `["pending", "approved", "completed"]`. ```yaml widget: - type: jellyseerr - url: http://jellyseerr.host.or.ip + type: seerr + url: http://seerr.host.or.ip key: apikeyapikeyapikeyapikeyapikey ``` diff --git a/public/locales/en/common.json b/public/locales/en/common.json index 142387984..738b14945 100644 --- a/public/locales/en/common.json +++ b/public/locales/en/common.json @@ -293,6 +293,7 @@ "pending": "Pending", "approved": "Approved", "available": "Available", + "completed": "Completed", "issues": "Open Issues" }, "overseerr": { diff --git a/src/widgets/seerr/component.jsx b/src/widgets/seerr/component.jsx index 9f88c7838..18da887b5 100644 --- a/src/widgets/seerr/component.jsx +++ b/src/widgets/seerr/component.jsx @@ -3,12 +3,13 @@ import Container from "components/services/widget/container"; import useWidgetAPI from "utils/proxy/use-widget-api"; -export const seerrDefaultFields = ["pending", "approved", "available"]; +export const seerrDefaultFields = ["pending", "approved", "completed"]; +const MAX_ALLOWED_FIELDS = 4; export default function Component({ service }) { const { widget } = service; - widget.fields = widget?.fields?.length ? widget.fields : seerrDefaultFields; + widget.fields = widget?.fields?.length ? widget.fields.slice(0, MAX_ALLOWED_FIELDS) : seerrDefaultFields; const isIssueEnabled = widget.fields.includes("issues"); const { data: statsData, error: statsError } = useWidgetAPI(widget, "request/count"); @@ -23,16 +24,24 @@ export default function Component({ service }) { + ); } + if (statsData.completed === undefined) { + // Newer versions added "completed", fallback to available + widget.fields = widget.fields.filter((field) => field !== "completed"); + widget.fields.push("available"); + } + return ( + ); diff --git a/src/widgets/seerr/component.test.jsx b/src/widgets/seerr/component.test.jsx index 5d2e63c48..a05687a00 100644 --- a/src/widgets/seerr/component.test.jsx +++ b/src/widgets/seerr/component.test.jsx @@ -28,23 +28,41 @@ describe("widgets/seerr/component", () => { expect(container.querySelectorAll(".service-block")).toHaveLength(3); expect(screen.getByText("seerr.pending")).toBeInTheDocument(); expect(screen.getByText("seerr.approved")).toBeInTheDocument(); - expect(screen.getByText("seerr.available")).toBeInTheDocument(); + expect(screen.getByText("seerr.completed")).toBeInTheDocument(); + expect(screen.queryByText("seerr.available")).toBeNull(); expect(screen.queryByText("seerr.issues")).toBeNull(); }); it("renders issues when enabled (and calls the issue/count endpoint)", () => { useWidgetAPI - .mockReturnValueOnce({ data: { pending: 1, approved: 2, available: 3 }, error: undefined }) + .mockReturnValueOnce({ data: { pending: 1, approved: 2, available: 3, completed: 4 }, error: undefined }) .mockReturnValueOnce({ data: { open: 1, total: 2 }, error: undefined }); const service = { - widget: { type: "seerr", url: "http://x", fields: ["pending", "approved", "available", "issues"] }, + widget: { type: "seerr", url: "http://x", fields: ["pending", "approved", "completed", "issues"] }, }; const { container } = renderWithProviders(, { settings: { hideErrors: false } }); expect(useWidgetAPI.mock.calls[1][1]).toBe("issue/count"); expect(container.querySelectorAll(".service-block")).toHaveLength(4); expect(screen.getByText("1 / 2")).toBeInTheDocument(); + expect(screen.getByText("4")).toBeInTheDocument(); + }); + + it("falls back from completed to available on older Seerr responses", () => { + useWidgetAPI + .mockReturnValueOnce({ data: { pending: 1, approved: 2, available: 3 }, error: undefined }) + .mockReturnValueOnce({ data: undefined, error: undefined }); + + const service = { + widget: { type: "seerr", url: "http://x", fields: ["pending", "approved", "completed"] }, + }; + + renderWithProviders(, { settings: { hideErrors: false } }); + + expect(service.widget.fields).toEqual(["pending", "approved", "available"]); + expect(screen.getByText("3")).toBeInTheDocument(); + expect(screen.queryByText("seerr.completed")).toBeNull(); }); it("renders error UI when issues are enabled and issue/count errors", () => { diff --git a/src/widgets/seerr/widget.test.js b/src/widgets/seerr/widget.test.js index 14ffdfca9..cdcda3a8f 100644 --- a/src/widgets/seerr/widget.test.js +++ b/src/widgets/seerr/widget.test.js @@ -4,7 +4,7 @@ import { expectWidgetConfigShape } from "test-utils/widget-config"; import widget from "./widget"; -describe("jellyseerr widget config", () => { +describe("seerr widget config", () => { it("exports a valid widget config", () => { expectWidgetConfigShape(widget); });