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

add YT subscribers #68

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

add YT subscribers #68

wants to merge 1 commit into from

Conversation

gianpaj
Copy link

@gianpaj gianpaj commented Oct 15, 2024

did it with a little script

https://github.com/gianpaj/youtube-md-subs

I still need to make it handle links with /user/ like:
https://www.youtube.com/user/procdalsinazev

@lucasricci
Copy link
Collaborator

Hello, thanks for your contribution!
I apologize for the time to answer you, I just got a few questions. While it might be a nice addition to the repo, it would need to be run at least once a week or something similar to keep it accurate. And the ´/user´ problem should be dealt with. What are your thoughts @naveen521kk?
Sincerely,
Lucas

@naveen521kk
Copy link
Member

Hmm, no particular comments about this one. I do agree that it should be run at least once in a while so that it's updated.

Also make sure not to break manim-website-api while adding this.

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.

3 participants