-
Notifications
You must be signed in to change notification settings - Fork 26
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
Remove comment scraper #84
Remove comment scraper #84
Conversation
usage/local-api.md
Outdated
@@ -30,8 +30,7 @@ The Local API is one method of obtaining data from YouTube. For another method o | |||
| ---------------------------------------------------------------------------------------- | ------- | ----------------------------------------------------------------------------------------------------------------- | -------------------------------- | | |||
| [youtubei.js](https://github.com/LuanRT/YouTube.js) | MIT | Search Functionality + Suggestions, Playlists, Trending, Obtain Video Information, Generate DASH Files, Live Chat | No | | |||
| [videojs-vtt-thumbnails-freetube](https://github.com/FreeTubeApp/videojs-vtt-thumbnails) | MIT | Handle Video Thumbnails / Storyboards | Yes | | |||
| [yt-channel-info](https://github.com/FreeTubeApp/yt-channel-info) | ISC | Channel Info / Search | Yes | | |||
| [yt-comment-scraper](https://github.com/FreeTubeApp/yt-comment-scraper) | GPL-3.0 | Comment Info / Sort | Yes | | |||
| [yt-channel-info](https://github.com/FreeTubeApp/yt-channel-info) | ISC | Channel Info / Search | Yes | | Yes | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Big oof indeed. Is it possible for u to commit to this pr to fix it? I already turned off my machine and i am in snorlax mode
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -30,7 +30,7 @@ The Local API is one method of obtaining data from YouTube. For another method o | |||
| ---------------------------------------------------------------------------------------- | ------- | ----------------------------------------------------------------------------------------------------------------- | -------------------------------- | | |||
| [youtubei.js](https://github.com/LuanRT/YouTube.js) | MIT | Search Functionality + Suggestions, Playlists, Trending, Obtain Video Information, Generate DASH Files, Live Chat | No | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Forgot to add Comments to this line 😆
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you mean adding fetching comments to yt.js?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I was mostly joking though because we'll probably make that line a lot smaller once we migrate from yt-channel-info
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Haha okay i initially thought that u were joking but then my head said but maybe it isnt a joke?
So yeah my brain playing tricks on me😅
Should i create a pr for adding the comments?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I dont think it's necessary 🙂
Related FreeTubeApp/FreeTube#3072