Skip to content

Ensure component only patches are not lost due to locked tree#3942

Merged
SteffenDE merged 2 commits intomainfrom
sd-component-only-locked
Aug 13, 2025
Merged

Ensure component only patches are not lost due to locked tree#3942
SteffenDE merged 2 commits intomainfrom
sd-component-only-locked

Conversation

@SteffenDE
Copy link
Collaborator

Closes #3941. Possibly related to #3614.

When a component-only diff was applied, we did not account for the component being part of a locked tree. Thus, when the tree was unlocked, the component content was unintentionally restored to a previous state.

We solve this by checking for a locked tree and applying the patch to that instead.

Closes #3941.
Possibly related to #3614.

When a component-only diff was applied, we did not account for the
component being part of a locked tree. Thus, when the tree was unlocked,
the component content was unintentionally restored to a previous state.

We solve this by checking for a locked tree and applying the patch
to that instead.
@SteffenDE SteffenDE requested a review from chrismccord August 11, 2025 13:19
@SteffenDE SteffenDE merged commit 7d7b24e into main Aug 13, 2025
14 checks passed
@SteffenDE SteffenDE deleted the sd-component-only-locked branch August 13, 2025 15:53
SteffenDE added a commit that referenced this pull request Aug 13, 2025
* Ensure component only patches are not lost due to locked tree

Closes #3941.
Possibly related to #3614.

When a component-only diff was applied, we did not account for the
component being part of a locked tree. Thus, when the tree was unlocked,
the component content was unintentionally restored to a previous state.

We solve this by checking for a locked tree and applying the patch
to that instead.

* link to pr
SteffenDE added a commit that referenced this pull request Aug 20, 2025
SteffenDE added a commit that referenced this pull request Aug 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Async assigns combined with on mount hook causes inconsistent rendering

1 participant

Comments