Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: season/episode list on series details #2967

Merged
merged 9 commits into from
Aug 24, 2022
Merged

Conversation

sct
Copy link
Owner

@sct sct commented Aug 23, 2022

Description

This PR adds a season list (with availability status) and episode details to the series detail pages

Screenshot (if UI-related)

image

image

To-Dos

  • Successful build yarn build
  • Translation keys yarn i18n:extract
  • Database migration (if required)

Issues Fixed or Closed

  • Fixes #XXXX

@cypress
Copy link

cypress bot commented Aug 23, 2022



Test summary

24 0 0 0Flakiness 1


Run details

Project overseerr
Status Passed
Commit 390cd5a
Started Aug 24, 2022 4:05 AM
Ended Aug 24, 2022 4:09 AM
Duration 03:25 💡
OS Linux Ubuntu - 20.04
Browser Electron 102

View run in Cypress Dashboard ➡️


Flakiness

cypress/e2e/user/profile.cy.ts Flakiness
1 User Profile > loads plex watchlist

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

Copy link
Collaborator

@TheCatLady TheCatLady left a comment

Choose a reason for hiding this comment

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

Looks great! Just a couple of comments.

Also, another minor thing -- I feel like there's a bit of visual disconnect between the season heading and the episode list. I think it should appear as one element that gets expanded (e.g., they should connect instead of having space between) instead of a button and some stuff that appears below.

image

src/components/TvDetails/Season/index.tsx Outdated Show resolved Hide resolved
src/components/TvDetails/Season/index.tsx Show resolved Hide resolved
@sct sct enabled auto-merge (squash) August 24, 2022 03:58
@sct sct merged commit 8a2acb7 into develop Aug 24, 2022
@sct sct deleted the feature/episode-list branch August 24, 2022 04:09
@github-actions
Copy link

🎉 This PR is included in version 1.30.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants