Skip to content

Integration: move files to ts (jquery)#31648

Merged
anna-shakhova merged 2 commits into
DevExpress:25_2from
anna-shakhova:25_2__move_ts_integration
Nov 13, 2025
Merged

Integration: move files to ts (jquery)#31648
anna-shakhova merged 2 commits into
DevExpress:25_2from
anna-shakhova:25_2__move_ts_integration

Conversation

@anna-shakhova

Copy link
Copy Markdown
Contributor

No description provided.

@anna-shakhova anna-shakhova self-assigned this Nov 10, 2025
@anna-shakhova anna-shakhova force-pushed the 25_2__move_ts_integration branch 3 times, most recently from 6e45aa6 to 88de030 Compare November 12, 2025 10:10
@anna-shakhova anna-shakhova force-pushed the 25_2__move_ts_integration branch from 88de030 to 2933768 Compare November 12, 2025 14:42
@anna-shakhova anna-shakhova changed the title Integration: move files to ts Integration: move files to ts (jquery) Nov 13, 2025
@anna-shakhova anna-shakhova marked this pull request as ready for review November 13, 2025 08:48
Copilot AI review requested due to automatic review settings November 13, 2025 08:48

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 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 .js to .ts with proper TypeScript syntax
  • Updated test files to use ES6 imports instead of AMD/require
  • Updated import paths to reflect new __internal location
  • 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

Comment thread packages/devextreme/js/__internal/integration/jquery/hooks.ts
Comment thread packages/devextreme/js/__internal/integration/jquery/deferred.ts
@anna-shakhova anna-shakhova merged commit 8c683d0 into DevExpress:25_2 Nov 13, 2025
324 of 337 checks passed
@anna-shakhova anna-shakhova deleted the 25_2__move_ts_integration branch November 13, 2025 14:36
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.

3 participants