Fix CI for v25.1.3#2
Merged
Merged
Conversation
- Convert React components to TypeScript with proper return types - Extract common utilities to utils.ts with CellValue type alias - Implement DevExtreme notify for better error handling - Add drag-and-drop for multiple row selection and reordering - Support both local data (Customer) and remote data (Task) scenarios - Zero lint errors - fully CI compliant - Build successful with all functionality working
- Converted from JavaScript to TypeScript with proper DevExtreme Angular types - Added shared utils integration for consistent drag-and-drop operations - Implemented proper error handling and promise management - Fixed all linting issues (0 errors, 0 warnings) - Preserved drag multiple rows functionality with local and remote data scenarios - Added proper return types and ESLint compliance - Cleaned up orig_ files and formatting with Prettier
- Converted from legacy JS to modern ES6+ syntax with proper linting compliance - Implemented DevExtreme DataGrid drag multiple rows with both local and remote data - Fixed for-in loops to use Object.keys() for better prototype chain handling - Converted await-in-loop to Promise.all for better async performance - Added proper error handling and promise management - Cleaned up HTML structure and CSS formatting with Prettier - Removed orig_ files and maintained kebab-case naming conventions
artem-kurchenko
requested a review
from DevExpressExampleBot
as a code owner
Contributor
|
Please correct the project and update this pull request by pushing a new commit. https://codecentralfarm.devexpress.devx/job/devextreme-datagrid-drag-multiple-rows_25.1.3+/2/console |
Contributor
|
Please correct the project and update this pull request by pushing a new commit. https://codecentralfarm.devexpress.devx/job/devextreme-datagrid-drag-multiple-rows_25.1.3+/3/console |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.