[PR #1050] [MERGED] Remove YoutubeLoader dependency #3561

Closed
opened 2026-02-22 18:34:02 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/Mintplex-Labs/anything-llm/pull/1050
Author: @timothycarambat
Created: 4/5/2024
Status: Merged
Merged: 4/5/2024
Merged by: @timothycarambat

Base: masterHead: 1049-remove-yt-dep


📝 Commits (9)

  • 7dcb7e6 WIP data connector redesign
  • a94514d new UI for data connectors complete
  • 46a346d remove old data connector page/cleanup imports
  • 2d0bd9c Merge branch 'master' into 1032-feat-implement-data-connector-redesign
  • e618fd7 Merge branch 'master' of github.com:Mintplex-Labs/anything-llm into 1032-feat-implement-data-connector-redesign
  • dc7c156 cleanup of UI and imports
  • 7b19358 Remove Youtube Transcript dep and move in-house
  • 0179d5b Merge branch 'master' of github.com:Mintplex-Labs/anything-llm into 1049-remove-yt-dep
  • 0ff04a8 lang pref default to en

📊 Changes

6 files changed (+257 additions, -34 deletions)

View changed files

📝 collector/package.json (+2 -2)
collector/utils/extensions/YoutubeTranscript/YoutubeLoader/index.js (+90 -0)
collector/utils/extensions/YoutubeTranscript/YoutubeLoader/youtube-transcript.js (+115 -0)
📝 collector/utils/extensions/YoutubeTranscript/index.js (+5 -7)
📝 collector/yarn.lock (+36 -19)
📝 frontend/src/components/SettingsSidebar/index.jsx (+9 -6)

📄 Description

Pull Request Type

  • feat
  • 🐛 fix
  • ♻️ refactor
  • 💄 style
  • 🔨 chore
  • 📝 docs

Relevant Issues

resolves #1049

What is in this change?

Breakout YouTube loader and dependencies for more control and hot-fixes.

Additional Information

Add any other context about the Pull Request here that was not captured above.

Developer Validations

  • I ran yarn lint from the root of the repo & committed changes
  • Relevant documentation has been updated
  • I have tested my code functionality
  • Docker build succeeds locally

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/Mintplex-Labs/anything-llm/pull/1050 **Author:** [@timothycarambat](https://github.com/timothycarambat) **Created:** 4/5/2024 **Status:** ✅ Merged **Merged:** 4/5/2024 **Merged by:** [@timothycarambat](https://github.com/timothycarambat) **Base:** `master` ← **Head:** `1049-remove-yt-dep` --- ### 📝 Commits (9) - [`7dcb7e6`](https://github.com/Mintplex-Labs/anything-llm/commit/7dcb7e698f60d693fa573e2fb25a16764c5b66a7) WIP data connector redesign - [`a94514d`](https://github.com/Mintplex-Labs/anything-llm/commit/a94514d283a84c3549d289be8cf9cdbe9eff9eb4) new UI for data connectors complete - [`46a346d`](https://github.com/Mintplex-Labs/anything-llm/commit/46a346d72a011d1de5004fb8a7482c1d4d17b0a4) remove old data connector page/cleanup imports - [`2d0bd9c`](https://github.com/Mintplex-Labs/anything-llm/commit/2d0bd9c7ff039efd7bf550e0a5a57222c8509fd0) Merge branch 'master' into 1032-feat-implement-data-connector-redesign - [`e618fd7`](https://github.com/Mintplex-Labs/anything-llm/commit/e618fd79cbe2a5091e7c612028835c6b8e7c08c9) Merge branch 'master' of github.com:Mintplex-Labs/anything-llm into 1032-feat-implement-data-connector-redesign - [`dc7c156`](https://github.com/Mintplex-Labs/anything-llm/commit/dc7c1569048d7fa37eea172ac1f1088e082151da) cleanup of UI and imports - [`7b19358`](https://github.com/Mintplex-Labs/anything-llm/commit/7b1935892dc3539bcfb3d8c018c3fa91cef0bbb4) Remove Youtube Transcript dep and move in-house - [`0179d5b`](https://github.com/Mintplex-Labs/anything-llm/commit/0179d5b8671df43c0905b531968e6e9563c61675) Merge branch 'master' of github.com:Mintplex-Labs/anything-llm into 1049-remove-yt-dep - [`0ff04a8`](https://github.com/Mintplex-Labs/anything-llm/commit/0ff04a83ff32cf6a65016e10224efc173b144d3d) lang pref default to en ### 📊 Changes **6 files changed** (+257 additions, -34 deletions) <details> <summary>View changed files</summary> 📝 `collector/package.json` (+2 -2) ➕ `collector/utils/extensions/YoutubeTranscript/YoutubeLoader/index.js` (+90 -0) ➕ `collector/utils/extensions/YoutubeTranscript/YoutubeLoader/youtube-transcript.js` (+115 -0) 📝 `collector/utils/extensions/YoutubeTranscript/index.js` (+5 -7) 📝 `collector/yarn.lock` (+36 -19) 📝 `frontend/src/components/SettingsSidebar/index.jsx` (+9 -6) </details> ### 📄 Description ### Pull Request Type <!-- For change type, change [ ] to [x]. --> - [ ] ✨ feat - [x] 🐛 fix - [ ] ♻️ refactor - [ ] 💄 style - [ ] 🔨 chore - [ ] 📝 docs ### Relevant Issues <!-- Use "resolves #xxx" to auto resolve on merge. Otherwise, please use "connect #xxx" --> resolves #1049 ### What is in this change? Breakout YouTube loader and dependencies for more control and hot-fixes. ### Additional Information Add any other context about the Pull Request here that was not captured above. ### Developer Validations <!-- All of the applicable items should be checked. --> - [x] I ran `yarn lint` from the root of the repo & committed changes - [x] Relevant documentation has been updated - [x] I have tested my code functionality - [x] Docker build succeeds locally --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-22 18:34:02 -05:00
yindo closed this issue 2026-02-22 18:34:02 -05:00
yindo changed title from [PR #1050] Remove YoutubeLoader dependency to [PR #1050] [MERGED] Remove YoutubeLoader dependency 2026-06-05 15:14:04 -04:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Mintplex-Labs/anything-llm#3561