Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

patch(vest): Set history root when exiting the root node #1153

Merged
merged 1 commit into from
Aug 12, 2024

Conversation

ealush
Copy link
Owner

@ealush ealush commented Aug 12, 2024

In the previous version of this code, Vest would set the history root node right when entering the topmost root, making the concept of "history" root node meaningless. This correctly sets the history root node only when exiting the topmost node.

Copy link

vercel bot commented Aug 12, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
vest ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 12, 2024 8:34pm
vest-next ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 12, 2024 8:34pm

@ealush ealush force-pushed the 2024-08-11-put-history-in-the-past branch from 3841ee9 to ec3f882 Compare August 12, 2024 20:27
@ealush ealush marked this pull request as ready for review August 12, 2024 20:29
@ealush ealush changed the title 2024 08 11 put history in the past patch(vest): Set history root when exiting the root node Aug 12, 2024
@ealush ealush merged commit a48e91b into latest Aug 12, 2024
4 of 6 checks passed
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.

1 participant