Skip to content
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.

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

All Yt Video thumbnail is Visible #307

Closed
wants to merge 3 commits into from

Conversation

divyansh-prks
Copy link
Contributor

@divyansh-prks divyansh-prks commented Feb 11, 2025

Description

Screencast from 2025-02-12 01-52-25.webm
I have done changes in the directory microcks.io/content/events/_index.md , where instead of rendering all yt videos using iframe i have used anchored tag and link thumbnails of all the videos which when user clicks on those thumbnail will play youtube video.

It enhances the performance and also it shows all yt video are playable correctly .

resolves issue #293

Related issue(s)

Copy link

👋 @divyansh-prks

Welcome to the Microcks community! 💖

Thanks and congrats 🎉 for opening your first pull request here! Be sure to follow the pull request template or please update it accordingly.

Hope you have a great time there!

🌟 ~~~~~~~~~ 🌟

📢 If you like Microcks, please ⭐ star ⭐ our repo to support it!

🙏 It really helps the project to gain momentum and credibility. It's a small contribution back to the project with a big impact.

@yada
Copy link
Member

yada commented Feb 11, 2025

Description

  • ...
  • ...
  • ...

Related issue(s)

Please add some infos and do not forget the related issue number (important to track and follow) ;)

Signed-off-by: divyansh <[email protected]>
@yada
Copy link
Member

yada commented Feb 11, 2025

@divyansh-prks "This branch has conflicts that must be resolved" as config/_default/hugo.toml has been changed in the meantime. Please update your PR accordingly, see: githubteacher/github-for-developers-sept-2015#648

@divyansh-prks
Copy link
Contributor Author

yes i did changes in hugo.toml file because i wanted to solves all the checks by myself . I will make sure i will resolved it as soon as possible

@yada
Copy link
Member

yada commented Feb 11, 2025

yes i did changes in hugo.toml file because i wanted to solves all the checks by myself . I will make sure i will resolved it as soon as possible

Thanks cause check fixed already with #302

@yada
Copy link
Member

yada commented Feb 11, 2025

BTW do not mix different changes in the same PR when it is linked to different issues.
For content/documentation/guides/automation/cli.md we have #309

@yada
Copy link
Member

yada commented Feb 11, 2025

BTW do not mix different changes in the same PR when it is linked to different issues. For content/documentation/guides/automation/cli.md we have #309

This fix is merged, so now you also need to remove content/documentation/guides/automation/cli.md from this PR
and hugo.toml 😎🙌

@divyansh-prks
Copy link
Contributor Author

divyansh-prks commented Feb 11, 2025

hey @yada i was getting some problem to fix the hugo.toml conflict so i created another branch and solved #307 & #298 both issues . You can check the pr #311

@yada
Copy link
Member

yada commented Feb 11, 2025

hey @yada i was getting some problem to fix the hugo.toml conflict so i created another branch and solved #307 & #298 both issues . You can check the pr #311

I know these conflict issues are tricky. I'll merge this new PR, but for the next PR, please see the git command to keep the same PR and resolve conflicts, as this is a very common use case on GitHub projects and something to learn: many other projects will not accept to manage conflicts by closing and creating new PRs 😎

@divyansh-prks
Copy link
Contributor Author

thank you for understanding yada . Yeah i was actually learning those commands right now XD

@yada yada closed this in #311 Feb 12, 2025
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