Skip to content

feat(api): add personalized to the related movie and show routes - #1055

Merged
vladjerca merged 1 commit into
masterfrom
feat/related-personalized
Nov 2, 2026
Merged

vladjerca merged 1 commit into
masterfrom
feat/related-personalized

Conversation

@vladjerca

Copy link
Copy Markdown
Contributor

Adds an optional personalized boolean to GET /movies/:id/related and GET /shows/:id/related.

When it is true and the caller is a VIP or director, the API returns the same related titles re-ordered toward the caller's taste. For everyone else the param is ignored and the list is unchanged.

  • New shared personalizedQuerySchema, merged into both related routes.
  • Route descriptions mention the param.

The server change that reads the param is in review and not live yet. Clients should not send it until it is, and the SDK publish waits for that too. Until then the API ignores the param.

The /related/smart aliases are not modeled in the SDK, so they have nothing to change here.

Test plan

  • bun run check and bun test in projects/api
  • bun run publish:check
  • bun run openapi:validate

@vladjerca
vladjerca marked this pull request as ready for review November 2, 2026 19:03
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-02T19:04:21.334011Z 76aa89d Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@vladjerca
vladjerca merged commit 090d333 into master Nov 2, 2026
15 checks passed
@vladjerca
vladjerca deleted the feat/related-personalized branch November 2, 2026 19:06
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.

1 participant