Integration: move files to ts (jquery)#31648
Merged
anna-shakhova merged 2 commits intoNov 13, 2025
Merged
Conversation
6e45aa6 to
88de030
Compare
88de030 to
2933768
Compare
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR migrates jQuery integration files from JavaScript to TypeScript, moving them from js/integration/jquery/ to js/__internal/integration/jquery/. The migration includes converting from AMD/require modules to ES6 imports and adding TypeScript type annotations.
Key Changes
- Migrated 10 jQuery integration files from
.jsto.tswith proper TypeScript syntax - Updated test files to use ES6 imports instead of AMD/require
- Updated import paths to reflect new
__internallocation - Main entry point now redirects to the new TypeScript implementation
Reviewed Changes
Copilot reviewed 28 out of 28 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
packages/devextreme/testing/tests/DevExpress.jquery/eventRegistrator.tests.js |
Converted from AMD to ES6 imports; replaced early return pattern with QUnit.module.if |
packages/devextreme/testing/tests/DevExpress.jquery/deferred.tests.js |
Converted from AMD to ES6 imports |
packages/devextreme/testing/tests/DevExpress.core/getPublicElement.JQuery.tests.js |
Updated import path to new __internal location |
packages/devextreme/testing/runner/Views/Main/RunSuite.cshtml |
Updated component_registrator path to __internal location |
packages/devextreme/js/integration/jquery/*.js |
Deleted original JavaScript files |
packages/devextreme/js/integration/jquery.js |
Now redirects to __internal implementation |
packages/devextreme/js/__internal/integration/jquery/*.ts |
New TypeScript implementations with ES6 imports |
pharret31
approved these changes
Nov 13, 2025
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
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.