Skip to content

Fix <FilterLiveForm> may override latest users inputs when they type at the same pace than the debounce delay#10952

Merged
djhi merged 2 commits intomasterfrom
fix-filter-live-form
Sep 23, 2025
Merged

Fix <FilterLiveForm> may override latest users inputs when they type at the same pace than the debounce delay#10952
djhi merged 2 commits intomasterfrom
fix-filter-live-form

Conversation

@djhi
Copy link
Contributor

@djhi djhi commented Sep 22, 2025

Problem

When they type at the same pace than the debounce delay set on List, they may have their inputs overridden by the latest debounced filter value from useListParams.

Solution

  • Ignore the next change on filterValues after a user modification.

How To Test

  • All tests should pass

Additional Checks

  • The PR targets master for a bugfix or a documentation fix, or next for a feature
  • The PR includes unit tests (if not possible, describe why)
  • The PR includes one or several stories (if not possible, describe why)
  • The documentation is up to date

Also, please make sure to read the contributing guidelines.

…e at the same pace than the debounce delay
@djhi djhi added the RFR Ready For Review label Sep 22, 2025
await new Promise(resolve => setTimeout(resolve, 500));
});

it('should work when users type at the exact same pace than the debounce delay', async () => {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Without the fix, this test is flaky

@djhi djhi merged commit 8df4f87 into master Sep 23, 2025
15 checks passed
@djhi djhi deleted the fix-filter-live-form branch September 23, 2025 12:12
@djhi djhi added this to the 5.11.4 milestone Sep 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

RFR Ready For Review

Development

Successfully merging this pull request may close these issues.

2 participants