Enhancement: jellyseer completed (#6329)

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)

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
 ```

public/locales/en/common.json

@@ -293,6 +293,7 @@
         "pending": "Pending",
         "approved": "Approved",
         "available": "Available",
+        "completed": "Completed",
         "issues": "Open Issues"
     },
     "overseerr": {

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 }) {
         <Block label="seerr.pending" />
         <Block label="seerr.approved" />
         <Block label="seerr.available" />
+        <Block label="seerr.completed" />
         <Block label="seerr.issues" />
       </Container>
     );
   }
 
+  if (statsData.completed === undefined) {
+    // Newer versions added "completed", fallback to available
+    widget.fields = widget.fields.filter((field) => field !== "completed");
+    widget.fields.push("available");
+  }
+
   return (
     <Container service={service}>
       <Block label="seerr.pending" value={statsData.pending} />
       <Block label="seerr.approved" value={statsData.approved} />
       <Block label="seerr.available" value={statsData.available} />
+      <Block label="seerr.completed" value={statsData.completed} />
       <Block label="seerr.issues" value={`${issueData?.open} / ${issueData?.total}`} />
     </Container>
   );

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(<Component service={service} />, { 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(<Component service={service} />, { 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", () => {

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);
   });