Skip to content

fix(vue-vtable): preserve immediate editor updates - #5328

Merged
fangsmile merged 3 commits into
developfrom
fix/issue-4925-datasource-editor
Sep 21, 2026
Merged

fangsmile merged 3 commits into
developfrom
fix/issue-4925-datasource-editor

Conversation

@fangsmile

Copy link
Copy Markdown
Contributor

Summary

  • keep dynamic editor completion synchronous unless user validation is asynchronous
  • prevent a completed editor from overwriting an immediate same-cell changeCellValue call
  • add regression coverage, an async-data-source Vue demo, and a patch changeset

Closes #4925

Test plan

  • Confirm the regression test fails before the fix with editor-value replacing button-value
  • Run Vue VTable tests: 2 suites, 14 tests
  • Run dependency-scope and pre-push package test suites successfully
  • Run Vue TypeScript compile and Rush dependency build
  • Run ESLint, Prettier, and git diff --check
  • Verify the demo in local Chrome: the cell remains approved with no console errors

🤖 Generated with Claude Code

Keep validation synchronous unless user validation is asynchronous.
This prevents completed editors from overwriting immediate same-cell updates.

Co-Authored-By: Claude Sonnet 4.6 noreply@anthropic.com
Preserve cleanup for synchronous validation errors and support cross-realm promise-like results.

Co-Authored-By: Claude Sonnet 4.6 noreply@anthropic.com
Convert thenable errors into rejected promises so editor validation state can recover.

Co-Authored-By: Claude Sonnet 4.6 noreply@anthropic.com
@fangsmile
fangsmile merged commit a4642ea into develop Sep 21, 2026
6 of 7 checks passed
@fangsmile
fangsmile deleted the fix/issue-4925-datasource-editor branch September 21, 2026 11:19
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.

[Question] 在使用dynamic-render-editor的场景(使用datasource)下,changeCellValue不工作

1 participant