Skip to content

test(vtable): cover filtered addRecords persistence - #5326

Merged
fangsmile merged 1 commit into
developfrom
fix/issue-4607-filtered-add-records
Sep 20, 2026
Merged

fangsmile merged 1 commit into
developfrom
fix/issue-4607-filtered-add-records

Conversation

@fangsmile

@fangsmile fangsmile commented Sep 20, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Closes #4607

Test plan

  • Verify the regression test fails on pre-fix commit a68f06035 because David and Eve disappear
  • Run rushx test --runInBand __tests__/data-update/listTable-data-events.test.ts (15 tests passed)
  • Run full rushx test --runInBand (80 suites, 387 tests passed)
  • Run rushx compile
  • Run full VTable source ESLint (0 errors; existing warnings only)
  • Run rush build --to @visactor/vtable
  • Verify the debug demo in Chrome at desktop and narrow viewports
  • Pass repository pre-push package tests

Bugserver note

The attempted photo case was removed because every available branch/build passed, so it could not distinguish the regression and was not valid evidence.

🤖 Generated with Claude Code

Preserve regression coverage for source-record synchronization after filters are cleared.

Co-Authored-By: Claude Sonnet 4.6 noreply@anthropic.com
@fangsmile
fangsmile merged commit d075507 into develop Sep 20, 2026
7 of 8 checks passed
@fangsmile
fangsmile deleted the fix/issue-4607-filtered-add-records branch September 20, 2026 11:13
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.

[Bug] vue-table 使用updateFilterRules后,使用addRecords添加数据未修改原records

1 participant