Skip to content

feat(discover): customize built-in Networks shelf - #241

Closed
snapetech wants to merge 1 commit into
mainfrom
codex/discover-networks-20261013
Closed

snapetech wants to merge 1 commit into
mainfrom
codex/discover-networks-20261013

Conversation

@snapetech

@snapetech snapetech commented Oct 13, 2026 •

Copy link
Copy Markdown
Collaborator

Description

Administrators can edit the built-in Networks shelf by adding TMDB network IDs, changing order, removing entries, restoring defaults, or saving an empty list. The editor requires a usable TMDB logo, and the server validates the complete list before storing it. Other Discover slider data keeps its existing size limit.

This adds the admin guide, API description, and release note.

AI assistance: OpenAI Codex assisted with implementation, documentation, and verification. The maintainer directed the scope and authorized the changes.

How Has This Been Tested?

  • pnpm exec vitest run server/lib/discoverNetworks.test.ts server/routes/settingsDiscover.test.ts server/routes/discover.openapi.test.ts — 34 tests passed.
  • pnpm build — passed; includes translation checks, current-batch and shared-style checks, and Next/server builds.
  • Focused ESLint and Prettier checks — passed.
  • pnpm attribution:check — passed.
  • Hosted validation on this head passed unit tests, all six Cypress shards, Playwright, CodeQL, translations, and build checks.
  • Cypress Networks scenario — passed, including add, reorder, save, reload, and reset. The full Discover customization spec had one intermittent local wait timeout; the exact-head hosted Cypress run subsequently passed all six shards.

Screenshots / Logs (if applicable)

Visually inspected the editor at desktop and 390×844. The form controls fit on one row at the narrow width. No screenshots or runtime data are included in the repository.

Release Notes

  • I added a release-note fragment under release-notes/.
  • This change is internal-only and does not need a user-facing release note.
  • The fragment includes audience, area, action, and breaking-change status.
  • I previewed the release text with pnpm release-notes:preview.

Checklist:

  • I have read and followed the contribution guidelines.
  • Disclosed use of AI and its scope.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • Successful build pnpm build.
  • Translation keys pnpm i18n:extract.
  • Database migration (if required).

@github-actions

Copy link
Copy Markdown

This pull request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged.

@snapetech snapetech closed this Oct 15, 2026
@snapetech
snapetech deleted the codex/discover-networks-20261013 branch October 16, 2026 01:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant