-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
feat(Tools): ✨ add YouTube Video transcript fetching #5034
feat(Tools): ✨ add YouTube Video transcript fetching #5034
Conversation
Hey @Yeuoly, do you think it's better to put this under the YouTube provider or have it as a separate provider |
I think it is impossible to put this under YouTube provider, as YouTube provider needs an authorization but this tool does not need it. |
This is indeed a problem. 🤔 |
Also wanted to do that until I found this out |
Yes, this is what I mentioned before. We might need a provider group to aggregate all brand-related providers, such as Google, SerpAPI, and Google Mail, together. |
Would be pleasure to collaborate in this field |
It looks like we need some UI design work on the frontend. On the backend, we just need to add a |
Do you mean backend or both? I notice that you have a very good UI developer or principals, I would like to not destroy that :) If you send me the UX project I can implement it both on front and back ends |
I think she's talking about the backend. If our API includes a 'brand' field, we can start designing and completing the frontend work. |
Description
Adding a tool for fetching transcript of a YouTube video.
Type of Change
How Has This Been Tested?
Tested locally
Suggested Checklist:
dev/reformat
(backend) andcd web && npx lint-staged
(frontend) to appease the lint godsoptional
I have made corresponding changes to the documentationoptional
I have added tests that prove my fix is effective or that my feature worksoptional
New and existing unit tests pass locally with my changes