search

imhardikdesai / Youtube-Clone

https://my-yt-lite.netlify.app/
2 stars 2 forks source link

Video have their channel image #6

Closed gandholi-kumar closed 12 months ago

gandholi-kumar commented 12 months ago

Hi @imhardikdesai Please review and approve my merge request.

Tasks Accomplished:

  1. Fetched the channel avatar dynamically and displayed it respectively as shown in the below snapshot. Authorized API_KEY: Light theme: image Dark theme: image

    Unauthorized API_KEY: When the API_KEY expires we will display the static data. Light theme: image Dark theme: image

  2. Modified the file src/components/UpNav/Upnav.jsx to address the unique key error displayed in the browser console.

  3. Fetching the top 40 mostPopular videos on page load, respective changes are made in src/components/Navbar/Navbar.jsx and src/context/YtState.js.

  4. Moved the API_KEY to the .env file.

  5. Created a util folder(src/utils/GetChannelIds.js) to handle Video and avatar-specific match logic.

netlify[bot] commented 12 months ago

Deploy Preview for my-yt-lite ready!

Name Link
Latest commit ec285c0f9ec0f4846f531979b38d2ffd10e625d8
Latest deploy log https://app.netlify.com/sites/my-yt-lite/deploys/652061872f554c00083d6898
Deploy Preview https://deploy-preview-6--my-yt-lite.netlify.app/
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

imhardikdesai commented 12 months ago

Hello @gandholi-kumar appreciate your work, can you take pull from main branch then I will merge your PR. Currently it have conflict with base branch

gandholi-kumar commented 12 months ago

Hello @gandholi-kumar appreciate your work, can you take pull from main branch then I will merge your PR. Currently it have conflict with base branch

Sure, will do it in a couple of minutes.

imhardikdesai commented 12 months ago

Looks good , thank you 😊