Skip to content

Conversation

@marker-dao
Copy link
Contributor

No description provided.

@marker-dao marker-dao requested a review from a team October 24, 2025 11:17
@marker-dao marker-dao self-assigned this Oct 24, 2025
@Copilot Copilot AI review requested due to automatic review settings October 24, 2025 11:17
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR refactors the FileUploader component by extracting inner classes and types into separate files, improving code organization and maintainability. The core FileUploader class remains in the main file while upload strategies, blob reading, and type definitions are modularized.

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
file_uploader.ts Removed inline class definitions and type declarations; now imports from separate modules
file_uploader.types.ts New file containing all TypeScript type definitions and interfaces previously inline
file_blob_reader.ts New file containing the FileBlobReader class extracted from main file
file_upload_strategy.base.ts New file containing the FileUploadStrategyBase abstract class
file_upload_strategy.chunks.base.ts New file containing the ChunksFileUploadStrategyBase class
file_upload_strategy.chunks.default.ts New file containing the DefaultChunksFileUploadStrategy class
file_upload_strategy.chunks.custom.ts New file containing the CustomChunksFileUploadStrategy class
file_upload_strategy.whole.base.ts New file containing the WholeFileUploadStrategyBase class
file_upload_strategy.whole.default.ts New file containing the DefaultWholeFileUploadStrategy class
file_upload_strategy.whole.custom.ts New file containing the CustomWholeFileUploadStrategy class

@marker-dao marker-dao merged commit 40f6f7f into DevExpress:25_2 Oct 24, 2025
151 of 155 checks passed
@marker-dao marker-dao deleted the 25_2_fu_grooming branch October 24, 2025 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants