Skip to content

London | 26-ITP-May | Tomislav Dukez | Sprint 3 | Project TV Show - #60

Open
tomdu3 wants to merge 78 commits into
CodeYourFuture:mainfrom
tomdu3:level-500
Open

London | 26-ITP-May | Tomislav Dukez | Sprint 3 | Project TV Show#60
tomdu3 wants to merge 78 commits into
CodeYourFuture:mainfrom
tomdu3:level-500

Conversation

@tomdu3

@tomdu3 tomdu3 commented Aug 13, 2026 •
edited
Loading

Copy link
Copy Markdown

Learners, PR Template

Self checklist

  • I have committed my files one by one, on purpose, and for a reason
  • I have titled my PR with REGION | COHORT_NAME | FIRST_NAME LAST_NAME | SPRINT NUM | PROJECT NAME
  • I have tested my changes
  • My changes follow the style guide
  • My changes meet the requirements of this task

Changelist

The changes satisfy the requirements:

  • Level-100: Episode Listing Display
  • Level-200: Live Episode Search
  • Level-300: Episode Selector Dropdown
  • Level-400: API Fetch and Show Selector
  • Level-500: Shows Listing Page and Caching, in detail:
    • Initial app load presents full listing of all shows.
    • Each show card displays name, image, summary, genres, status, rating, and runtime.
    • Clicking any show title switches to that show's episode view and hides shows listing.
    • Navigation button returns to shows listing and hides episode listing.
    • Free-text show search filters shows by name, summary, and genres.
    • Search and dropdown controls work cleanly when switching back and forth between views.
    • API response caching: no URL endpoint is fetched more than once per visit.

Netlify deployment link: https://cyf-tomdu3-tv.netlify.app/

tomdu3 and others added 30 commits July 30, 2026 19:37
@tomdu3
add my name and GH username to title
706994d
@tomdu3
Merge pull request #1 from tomdu3/level-0
a72fa85 
add my name and GH username to title
@tomdu3
add details div
a7a8d5f
@tomdu3
add display card function and example
c1db949
@tomdu3
add card template and remove details
9e7fb22
@tomdu3
fix redo displayCard to use template
7890ace
@tomdu3
add initial card styling
597c9ef
@tomdu3
add footer with TVMaze link and reference
9431400
@tomdu3
fix display all episodes
e82d9d9
@tomdu3
add styles for card shadows and footer
5f206e3
@tomdu3
Merge pull request #2 from tomdu3/level-100
df6130a 
Level 100 tasks
@edinakurdi
add alt text for images
ec91060
@edinakurdi
Update root element variable reference
5a4e878
@edinakurdi
Merge pull request #3 from tomdu3/level-200
146d7a1 
Level 200-refarcor
@edinakurdi
unpdate html
b76e8b5
@edinakurdi
implement search, add eventlistenet
a449b4d
@edinakurdi
Merge branch 'main' of https://github.com/tomdu3/Project-TV-Show into…
2bd2013 
… level-200
@edinakurdi
correct typos
f99ade1
@edinakurdi
update with a header tag to include the search bar and episode count
658656b
@edinakurdi
delete duplicate paragraph
dd7f406
@edinakurdi
add episode selector to html
f014618
@edinakurdi
add episode selector dropdown
b24ba4f
@edinakurdi
scroll to selected episode on dropdown
3dc0592
@edinakurdi
delete extensive comments
b70b047
@tomdu3
Merge pull request #4 from tomdu3/level-200
a47a537 
Level 200
@tomdu3
fix redesign header
4a3adf7
@tomdu3
fix header styling search and dropdown
3806bbb
@tomdu3
fix: implement mutually exclusive filtering via search bar and dropdo…
1b5420f 
…wn menu
@tomdu3
update episode selector placeholder, add search input hint, and adjus…
313576a 
…t footer layout
@tomdu3
improve layout responsiveness and accessibility with visually-hidden …
2ff3cd8 
…labels
edinakurdi and others added 27 commits August 12, 2026 16:03
@edinakurdi
add show selector event listener
54abfde
@edinakurdi
update show selector listener and handler -
499613d 
-check if the episode has been fetched yet
-handle errors
@edinakurdi
initialise page
5b1b1ef
@edinakurdi
fix grammar
c4d9f18
@edinakurdi
move DOM elemtnes to the top so they can be accessed later
0e72515
@edinakurdi
fix typos
e206caa
@tomdu3
Merge pull request #6 from tomdu3/level-400
bf08e87 
Level 400
@tomdu3
fix state initialisation of selectedShowId in state
83a4828
@tomdu3
refactor sort shows once upon fetch and selector population
cae31e6
@tomdu3
refactor remove redundant DOM get requests by using already defined g…
3718367 
…lobal el vars
@tomdu3
refactor rename root main element ID to cards-container
f358296
@tomdu3
add hourglass emoji to episode loading message
8f5f4fc
@tomdu3
add shows displaying details functionalities
ac1b2a7
@tomdu3
add template for the show cards
4abc6dd
@tomdu3
add changes for cards-container and show-card styles
f2ecc16
@tomdu3
add align-self: flex-start to card components
416bd5a
@tomdu3
add click event to show titles for episode selection and update styles
6ce2419
@tomdu3
add back-to-shows button and styles
cc081f6
@tomdu3
add functionality to toggle show/episode views with corresponding sea…
7b386cd 
…rch and buttons
@tomdu3
fix rename rating label and reorder show information fields in index.…
5d63d33 
…html
@tomdu3
add search filtering for shows by name, summary, and genres
2d6d67c
@tomdu3
add and display current show title above episode cards container
72c8ac1
@tomdu3
fix back to shows button style
7d8c7b0
@tomdu3
fix spacing for container and input elements for count display improv…
659a67a 
…ement
@tomdu3
add borders to show container and title
a74cba8
@tomdu3
Merge pull request #7 from tomdu3/level-500
d621942 
Level 500
@tomdu3
fix validation errors in html
60b2db0
@tomdu3 tomdu3 added Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. 🏕 Priority Mandatory This work is expected Module-Data-Flows labels Aug 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Module-Data-Flows Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. 🏕 Priority Mandatory This work is expected

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tomdu3 @edinakurdi