[PR #2965] [CLOSED] 2964 file type default #4185

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

📋 Pull Request Information

Original PR: https://github.com/Mintplex-Labs/anything-llm/pull/2965
Author: @wprice-uh
Created: 1/11/2025
Status: Closed

Base: masterHead: 2964-fileTypeDefault


📝 Commits (7)

  • e720a64 added ignore for .idea IDE files
  • 1b26858 checkpoint: settings gui works
  • a840884 updated processSingleFile to respond to fileTypeDefault
  • 4d4bc83 removing unused management gui
  • 756c37c cleaned up the code
  • 01315ca ran yarn lint
  • 29f5427 Merge branch 'Mintplex-Labs:master' into 2964-fileTypeDefault

📊 Changes

11 files changed (+185 additions, -2 deletions)

View changed files

📝 .gitignore (+2 -0)
📝 collector/processSingleFile/index.js (+2 -1)
frontend/src/models/experimental/fileTypeDefault.js (+24 -0)
📝 frontend/src/models/system.js (+2 -0)
frontend/src/pages/Admin/ExperimentalFeatures/Features/FileTypeDefault/toggle.jsx (+57 -0)
📝 frontend/src/pages/Admin/ExperimentalFeatures/features.js (+6 -0)
server/endpoints/experimental/fileTypeDefault.js (+59 -0)
📝 server/endpoints/experimental/index.js (+2 -0)
server/models/fileTypeDefault.js (+15 -0)
📝 server/models/systemSettings.js (+10 -0)
📝 server/utils/collectorApi/index.js (+6 -1)

📄 Description

Pull Request Type

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

Relevant Issues

resolves #2964

What is in this change?

Added an experimental setting labeled 'File Type Default' which, when turned on, changes the default behavior when a file being imported can't be recognized, from displaying an error message to treating the file as if it were plain text.

Additional Information

Happy to receive feedback and make changes as needed; I'm not well-versed in javascript or in the frameworks/libraries used here, so most of what I did was based on cloning the live-file-sync functionality.

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/2965 **Author:** [@wprice-uh](https://github.com/wprice-uh) **Created:** 1/11/2025 **Status:** ❌ Closed **Base:** `master` ← **Head:** `2964-fileTypeDefault` --- ### 📝 Commits (7) - [`e720a64`](https://github.com/Mintplex-Labs/anything-llm/commit/e720a64a2c45528bea63af5b42677b77a70441c8) added ignore for .idea IDE files - [`1b26858`](https://github.com/Mintplex-Labs/anything-llm/commit/1b26858e81269689848b9ac4bd29fc6d7d714614) checkpoint: settings gui works - [`a840884`](https://github.com/Mintplex-Labs/anything-llm/commit/a840884b44597318375cc9ff2e24ee1c1a2b5b93) updated processSingleFile to respond to fileTypeDefault - [`4d4bc83`](https://github.com/Mintplex-Labs/anything-llm/commit/4d4bc83bacfe0633c93dd58be4d1aaa32d64ccd4) removing unused management gui - [`756c37c`](https://github.com/Mintplex-Labs/anything-llm/commit/756c37c0f2076ab36e812f47879a3d843277a1d0) cleaned up the code - [`01315ca`](https://github.com/Mintplex-Labs/anything-llm/commit/01315cace1170cfdcbd1bd2a6b112f8f1dfc657d) ran yarn lint - [`29f5427`](https://github.com/Mintplex-Labs/anything-llm/commit/29f5427d8741667a06e217968a26e13ee9842db0) Merge branch 'Mintplex-Labs:master' into 2964-fileTypeDefault ### 📊 Changes **11 files changed** (+185 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `.gitignore` (+2 -0) 📝 `collector/processSingleFile/index.js` (+2 -1) ➕ `frontend/src/models/experimental/fileTypeDefault.js` (+24 -0) 📝 `frontend/src/models/system.js` (+2 -0) ➕ `frontend/src/pages/Admin/ExperimentalFeatures/Features/FileTypeDefault/toggle.jsx` (+57 -0) 📝 `frontend/src/pages/Admin/ExperimentalFeatures/features.js` (+6 -0) ➕ `server/endpoints/experimental/fileTypeDefault.js` (+59 -0) 📝 `server/endpoints/experimental/index.js` (+2 -0) ➕ `server/models/fileTypeDefault.js` (+15 -0) 📝 `server/models/systemSettings.js` (+10 -0) 📝 `server/utils/collectorApi/index.js` (+6 -1) </details> ### 📄 Description ### Pull Request Type - [x] ✨ feat - [ ] 🐛 fix - [ ] ♻️ refactor - [ ] 💄 style - [ ] 🔨 chore - [ ] 📝 docs ### Relevant Issues resolves #2964 ### What is in this change? Added an experimental setting labeled 'File Type Default' which, when turned on, changes the default behavior when a file being imported can't be recognized, from displaying an error message to treating the file as if it were plain text. ### Additional Information Happy to receive feedback and make changes as needed; I'm not well-versed in javascript or in the frameworks/libraries used here, so most of what I did was based on cloning the live-file-sync functionality. ### Developer Validations - [x] I ran `yarn lint` from the root of the repo & committed changes - [ ] 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:35:19 -05:00
yindo closed this issue 2026-02-22 18:35:19 -05:00
yindo changed title from [PR #2965] 2964 file type default to [PR #2965] [CLOSED] 2964 file type default 2026-06-05 15:17:16 -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#4185