Skip to content

Conversation

@mpreyskurantov
Copy link
Contributor

Copy link
Contributor

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 adds anti-forgery token support to the DataGrid BatchUpdateRequest demo across all supported frameworks (jQuery, Angular, React, Vue). The implementation fetches CSRF tokens from the backend and includes them in API requests to support anti-forgery validation. The demo is also added to the skipped visual tests list.

Changes:

  • Added anti-forgery token fetching and caching mechanism for all frameworks
  • Updated API endpoints to use the new BatchUpdateWebApi endpoint
  • Added BatchUpdateRequest to the list of skipped visual tests for all frameworks
  • Also updated the CollaborativeEditing jQuery demo with similar anti-forgery token support

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 12 comments.

Show a summary per file
File Description
apps/demos/utils/visual-tests/matrix-test-helper.ts Added BatchUpdateRequest to skipped visual tests for all frameworks
apps/demos/Demos/DataGrid/CollaborativeEditing/jQuery/index.js Updated to use anti-forgery tokens and fixed URL path consistency
apps/demos/Demos/DataGrid/BatchUpdateRequest/jQuery/index.js Implemented anti-forgery token support with jQuery-specific Deferred pattern
apps/demos/Demos/DataGrid/BatchUpdateRequest/Vue/App.vue Implemented anti-forgery token support using async/await with fetch API
apps/demos/Demos/DataGrid/BatchUpdateRequest/ReactJs/App.js Implemented anti-forgery token support using async/await with fetch API
apps/demos/Demos/DataGrid/BatchUpdateRequest/React/App.tsx TypeScript version of React implementation with proper typing
apps/demos/Demos/DataGrid/BatchUpdateRequest/Angular/app/app.service.ts New service providing token management via RxJS observables and HTTP interceptor
apps/demos/Demos/DataGrid/BatchUpdateRequest/Angular/app/app.component.ts Updated component to use the new token service and interceptor

@mpreyskurantov mpreyskurantov merged commit 4519473 into DevExpress:26_1 Jan 16, 2026
161 of 166 checks passed
@mpreyskurantov mpreyskurantov deleted the 26_1-32057 branch January 16, 2026 20:01
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