-
-
Notifications
You must be signed in to change notification settings - Fork 21.3k
feat: Add XLSX file upload support and fix file processing logic #4833
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
base: main
Are you sure you want to change the base?
Conversation
- Add XLSX support to availableFileTypes in FileUpload.jsx - Update file extension mapping: .xlsx now maps to 'file' instead of 'csvFile' - Add MIME type mapping for Excel files (application/vnd.ms-excel, application/vnd.openxmlformats-officedocument.spreadsheetml.sheet) - Fix critical bug in createAttachment.ts, buildChatflow.ts, upsertVector.ts: use fileInputFieldFromMimeType instead of fileInputFieldFromExt - Improve error logging in ChatMessage.jsx for better debugging - Enable XLSX files to be processed by File Loader with LoadSheet functionality
packages/components/src/utils.ts
Outdated
return 'csvFile' | ||
case '.xlsm': | ||
case '.xlsb': | ||
return 'file' |
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.
why are we changing this to file
?
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.
no need. I restored this lines.
@@ -371,7 +371,7 @@ export const executeFlow = async ({ | |||
if (fileInputFieldFromExt !== 'txtFile') { | |||
fileInputField = fileInputFieldFromExt | |||
} else if (fileInputFieldFromMimeType !== 'txtFile') { | |||
fileInputField = fileInputFieldFromExt | |||
fileInputField = fileInputFieldFromMimeType |
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.
why fileInputFieldFromExt
to fileInputFieldFromMimeType
?
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.
no need. I restored this lines.
π Summary
This PR adds comprehensive XLSX file upload support and fixes critical file processing bugs in Flowise.
β¨ What's Changed
XLSX Support Added
availableFileTypes
in FileUpload.jsx UI component.xlsx
,.xlsm
,.xlsb
now map to'csvFile'
application/vnd.ms-excel
application/vnd.openxmlformats-officedocument.spreadsheetml.sheet
Improvements
createAttachment.ts
with file detailsπ― Impact
π§ͺ Testing
π Benefits
Fixes potential file upload processing bugs and adds highly requested XLSX support.