London | 26-ITP-May | Zadri Abdule | Project TV Show | Level - 500 - #101
Open
Zadri415 wants to merge 41 commits into
Open
London | 26-ITP-May | Zadri Abdule | Project TV Show | Level - 500#101Zadri415 wants to merge 41 commits into
Zadri415 wants to merge 41 commits into
Conversation
London | 26-ITP-May | Zadri Abdule | TV Show Project | Level-200
London | 26-ITP-May | Zadri Abdule | Project TV Show | Level 400
London | 26-ITP-May | Zadri Abdule | Project TV Show | Level 400
London | 26-ITP-May | Vitalii Kmit | Project TV Show | Level 500
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Learners, PR Template
Self checklist
Changelist
Implemented the level 500 shows-listing feature:
Added a shows listing view on app start, showing name, image, summary, genres, status, and rating/runtime for all shows
Added free-text search across show name, genres, and summary
Clicking a show name fetches and displays its episodes (search + select still work as before), hiding the shows listing
Added a "Back to shows" control to return to the listing, hiding the episodes view
Episodes are cached per show so no show's episodes are fetched more than once per visit
Fixed a broken tag for episode cards that would have crashed the episodes view
Accessibility fixes: added alt text to all images, made the clickable show name a real so it's keyboard-focusable
Fixed episode summaries rendering as raw
tags instead of formatted HTML
Questions
N/A