Skip to content

Conversation

@eps1lon
Copy link
Collaborator

@eps1lon eps1lon commented Sep 10, 2025

Test case is in the storeStress-sync test which started failing after c97ec75

Disconnected elements will not be registered in the frontend so recording a reorder would then crash in the store with "Cannot reorder children for node "X" because no matching node was found in the Store.".

@meta-cla meta-cla bot added the CLA Signed label Sep 10, 2025
@github-actions github-actions bot added the React Core Team Opened by a member of the React Core Team label Sep 10, 2025
@eps1lon eps1lon marked this pull request as ready for review September 10, 2025 20:04
@eps1lon eps1lon requested a review from sebmarkbage September 10, 2025 20:04
Copy link
Collaborator

@sebmarkbage sebmarkbage left a comment

Choose a reason for hiding this comment

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

Could probably move this check to be on the recordResetChildren instead.

@eps1lon eps1lon force-pushed the sebbie/disconnected-reorder branch from 95f9679 to 7836f1f Compare September 11, 2025 16:34
@eps1lon eps1lon merged commit 7fc888d into facebook:main Sep 11, 2025
241 checks passed
@eps1lon eps1lon deleted the sebbie/disconnected-reorder branch September 11, 2025 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed React Core Team Opened by a member of the React Core Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants