Skip to content

Youtube PlayList Data Fetch#142

Merged
atapas merged 9 commits into
the-hustle-home-pagefrom
youtube-playlist-fetch
Apr 19, 2023
Merged

Youtube PlayList Data Fetch#142
atapas merged 9 commits into
the-hustle-home-pagefrom
youtube-playlist-fetch

Conversation

@Abhishek-90

Copy link
Copy Markdown

Fetching Video data from ReactPlay youtube Playlist.

@netlify

netlify Bot commented Apr 9, 2023

Copy link
Copy Markdown

Deploy Preview for activities-reactplay ready!

Name Link
🔨 Latest commit f49aca2
🔍 Latest deploy log https://app.netlify.com/sites/activities-reactplay/deploys/643fb2a7d0eb5d00080fffb5
😎 Deploy Preview https://deploy-preview-142--activities-reactplay.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 settings.

@atapas atapas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@Abhishek-90 don't commit the yarn lock file

Comment thread pages/index.js Outdated
@Abhishek-90

Copy link
Copy Markdown
Author

@Abhishek-90 don't commit the yarn lock file

Delete yarn.lock file

@atapas atapas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Provide a border radius and why the images are not fitting in the iframe?

image

@atapas atapas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks Good for Merge

@atapas atapas merged commit 7cf892e into the-hustle-home-page Apr 19, 2023
@atapas atapas deleted the youtube-playlist-fetch branch April 19, 2023 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants