Skip to content

Conversation

@waleedlatif1
Copy link
Collaborator

Summary

add more upload types to the file block

Type of Change

  • Bug fix

Testing

Tested manually.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel
Copy link

vercel bot commented Sep 19, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
sim Building Building Preview Comment Sep 19, 2025 5:07pm
1 Skipped Deployment
Project Deployment Preview Comments Updated (UTC)
docs Skipped Skipped Sep 19, 2025 5:07pm

@waleedlatif1 waleedlatif1 merged commit a1c518e into staging Sep 19, 2025
4 of 5 checks passed
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Greptile Summary

This PR expands the file upload capabilities of the file block by adding support for additional file types. The change modifies the acceptedTypes property in the file upload configuration from .pdf,.csv,.docx to .pdf,.csv,.doc,.docx,.txt,.md,.xlsx,.xls,.html,.htm,.pptx,.ppt. This enhancement aligns the frontend file upload restrictions with the backend's existing parsing capabilities.

The file block is a core component that allows users to either upload files directly or provide URLs to files. It includes sophisticated input method selection, file validation, and error handling. The backend already has dedicated parsers for all these file types (PdfParser, CsvParser, DocParser, DocxParser, TxtParser, MdParser, XlsxParser, PptxParser, and HtmlParser), so this change removes an artificial frontend restriction that was preventing users from uploading files the system could actually process.

This change improves the user experience by making all supported file formats available for upload, creating consistency between what the system can parse and what users can upload through the interface.

PR Description Notes:

  • The change is incorrectly categorized as a "Bug fix" in the Type of Change section when it's actually a feature enhancement

Confidence score: 5/5

  • This PR is safe to merge with minimal risk
  • Score reflects a simple configuration change that aligns frontend restrictions with existing backend capabilities
  • No files require special attention

1 file reviewed, no comments

Edit Code Review Bot Settings | Greptile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants