Skip to content

feat(likes): like series + My Shows view - #91

Merged
leopahl merged 1 commit into
mainfrom
feat/like-shows-my-shows
Jul 17, 2026
Merged

leopahl merged 1 commit into
mainfrom
feat/like-shows-my-shows

Conversation

@atharvamp

Copy link
Copy Markdown
Collaborator

What

Adds a Like feature: users can like series and see them in a new My Shows view.

Changes

Backend (user-progress, OpenAPI-first)

  • GET /user-progress/likes · PUT / DELETE /user-progress/likes/{seriesId}
  • Idempotent like via INSERT … ON CONFLICT DO NOTHING (race-safe)
  • Newest-liked-first ordering; per-user isolation

Frontend

  • My Shows page + nav link
  • ♥ toggle on cards, labeled Like button on the series detail page
  • useLikes hook: optimistic toggle, in-flight guard, order-preserving revert

Tests

  • Backend: 6 integration tests (like/list, idempotency, ordering, isolation, auth)
  • Frontend: useLikes + MyShowsPage (46 total green)
  • tsc, eslint, spec lint clean

Notes

  • LikesApi (Java) and TS types regenerate from api/openapi.yaml at build (gitignored, unchanged workflow)

@coderabbitai

coderabbitai Bot commented Jul 16, 2026

Copy link
Copy Markdown

Caution

Review failed

An error occurred during the review process. Please try again later.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/like-shows-my-shows

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@leopahl leopahl left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Really good work, thank you 🥇

@leopahl
leopahl merged commit 9fa1763 into main Jul 17, 2026
9 checks passed
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.

2 participants