-
Notifications
You must be signed in to change notification settings - Fork 665
Comments
Demos | DataGrid - BatchUpdateRequest: Support BackEnd / EndPoint Anti Forgery (all frameworks)#32057
Merged
mpreyskurantov merged 15 commits intoDevExpress:25_2 from Jan 16, 2026
Conversation
* Refactor anti-forgery token handling in BatchUpdateRequest demo - Fetch fresh CSRF token for each load and save operation - Use jQuery $.ajax() consistently instead of mixing with fetch API - Move error handling into helper functions (getAntiForgeryToken, sendBatchRequest) - Improve error messages with descriptive context - Use promise chaining with .then() for better readability - Remove stale global csrf variable * Refactor anti-forgery token handling to use caching
* Refactor anti-forgery token handling in BatchUpdateRequest demo - Fetch fresh CSRF token for each load and save operation - Use jQuery $.ajax() consistently instead of mixing with fetch API - Move error handling into helper functions (getAntiForgeryToken, sendBatchRequest) - Improve error messages with descriptive context - Use promise chaining with .then() for better readability - Remove stale global csrf variable * Refactor anti-forgery token handling to use caching * jq - improve error handling * react - implement csrf * react - refactoring * vue - implement csrf * angular - implement csrf
* Refactor anti-forgery token handling in BatchUpdateRequest demo - Fetch fresh CSRF token for each load and save operation - Use jQuery $.ajax() consistently instead of mixing with fetch API - Move error handling into helper functions (getAntiForgeryToken, sendBatchRequest) - Improve error messages with descriptive context - Use promise chaining with .then() for better readability - Remove stale global csrf variable * Refactor anti-forgery token handling to use caching * jq - improve error handling * react - implement csrf * react - refactoring * vue - implement csrf * angular - implement csrf * change folder structure * merge services in one file * app.service in root --------- Co-authored-by: Mikhail Preyskurantov <5574159+mpreyskurantov@users.noreply.github.com>
@mpreyskurantov
mpreyskurantov
requested a review
from a team
as a code owner
January 15, 2026 15:19
anna-shakhova
anna-shakhova
previously approved these changes
Jan 16, 2026
@mpreyskurantov
mpreyskurantov
changed the title
(削除) WIP: Demos | BackEnd / EndPoints anti forgery (削除ここまで)
(追記) Demos | DataGrid - BatchUpdateRequest: Support BackEnd / EndPoint Anti Forgery (all frameworks) (追記ここまで)
Jan 16, 2026
@mpreyskurantov
mpreyskurantov
dismissed
anna-shakhova’s stale review
via
January 16, 2026 13:08
f4ec259
anna-shakhova
anna-shakhova
approved these changes
Jan 16, 2026
@mpreyskurantov
mpreyskurantov
merged commit Jan 16, 2026
bb94682
into
DevExpress:25_2
276 of 289 checks passed
@mpreyskurantov
mpreyskurantov
deleted the
25_2-endpoints-anti-forgery-dev
branch
January 16, 2026 15:27
mpreyskurantov
added a commit
to mpreyskurantov/DevExtreme
that referenced
this pull request
Jan 16, 2026
...i Forgery (all frameworks) (DevExpress#32057) Co-authored-by: Tom <22076961+artem-kurchenko@users.noreply.github.com>
mpreyskurantov
added a commit
to mpreyskurantov/DevExtreme
that referenced
this pull request
Jan 16, 2026
...i Forgery (all frameworks) (DevExpress#32057) Co-authored-by: Tom <22076961+artem-kurchenko@users.noreply.github.com> # Conflicts: # apps/demos/Demos/DataGrid/BatchUpdateRequest/Angular/app/app.component.ts # apps/demos/utils/visual-tests/matrix-test-helper.ts
This was referenced Jan 16, 2026
mpreyskurantov
added a commit
to mpreyskurantov/DevExtreme
that referenced
this pull request
Feb 11, 2026
...i Forgery (all frameworks) (DevExpress#32057) Co-authored-by: Tom <22076961+artem-kurchenko@users.noreply.github.com> # Conflicts: # apps/demos/Demos/DataGrid/BatchUpdateRequest/Angular/app/app.component.ts # apps/demos/Demos/DataGrid/BatchUpdateRequest/Vue/App.vue # apps/demos/Demos/DataGrid/CollaborativeEditing/jQuery/index.js # apps/demos/utils/visual-tests/matrix-test-helper.ts
mpreyskurantov
added a commit
to mpreyskurantov/DevExtreme
that referenced
this pull request
Feb 16, 2026
...oint Anti Forgery (all frameworks) (DevExpress#32057)" This reverts commit bb94682. # Conflicts: # apps/demos/utils/visual-tests/matrix-test-helper.ts
mpreyskurantov
added a commit
to mpreyskurantov/DevExtreme
that referenced
this pull request
Feb 17, 2026
...oint Anti Forgery (all frameworks) (DevExpress#32057)" This reverts commit bb94682. # Conflicts: # apps/demos/utils/visual-tests/matrix-test-helper.ts
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.
Uh oh!
There was an error while loading. Please reload this page.
Cherry-picks
Skip From Testing (not in domain):
/apps/demos/testing/skipped-tests.js -> /apps/demos/utils/visual-tests/matrix-test-helper.ts