Skip to content

Challenge 3: hooks #51

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.

Sign up for GitHub

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

Jump to bottom
Draft
Miri-AP wants to merge 9 commits into
base: master
Choose a base branch
from
Draft

Challenge 3: hooks #51

Miri-AP wants to merge 9 commits into from

Conversation

@Miri-AP
Copy link

@Miri-AP Miri-AP commented Apr 7, 2021 •
edited
Loading

Acceptance Criteria

  • Videos in the Home View are fetched from the YouTube API and displayed correctly.
  • Users can search for YouTube videos using the search field on the header.
  • A video can be played from the Video Details View after clicking on it.
  • The video information and related videos list are displayed correctly on the Video Details View.
  • When a user clicks on a related video the video player, information and related videos list are updated.

Bonus Points

  • Custom Hooks for API calls are implemented and tested correctly.
  • Testing coverage is above 60%. (Please include a screenshot of the code coverage report in your PR description).

Screen Shot 2021-04-07 at 0 11 52

Miri-AP and others added 9 commits March 30, 2021 22:16
@Miri-AP
React challenge 1 header n video (#1)
065423b 
* Challenge 1: Add header style

* Use styled-components lib to style Header

* Add video card w/o style

* Add video card style and order info

* Update responsive view

* Answer Mini-Challenge 2 questions

Co-authored-by: miriam-alvarezp <[email protected]>
Mini-Challenge 3: Init Player page
9055083
Move header test and add Search to input
17a28d4
Update videoCard to handle click and test
d418b21
add video state hooks and test
7381c71
Add VideoPlayer and update videoCard
5e585b1
Fix lint
c67a498
Add default search and get videos from youtube api
0309f18
Fix lint and update test
3baa1b4
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

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Miri-AP