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

Concurrent Mode compatibility? πŸ˜ƒ #45

Open
sesam opened this issue Jul 13, 2021 · 1 comment
Open

Concurrent Mode compatibility? πŸ˜ƒ #45

sesam opened this issue Jul 13, 2021 · 1 comment

Comments

@sesam
Copy link

sesam commented Jul 13, 2021

In an older version of https://github.com/dai-shi/will-this-react-global-state-work-in-concurrent-rendering we had klyva included. At that time, klyva was passing all tests except this one:

check with useTransition
  βœ• check 7: proper branching with transition (6127 ms)

What is this test 7 checking? Is this something that is hard to fix in klyva? If it needs a de-optimisation, could it be as an optional extra or a plugin or monkeypatch under a contrib/ folder, so that klyva could be the first to pass all the known tests for concurrent mode? πŸŽ‰

@merisbahti
Copy link
Owner

Yeah that's a fun question, I haven't really seen any other (feature rich) state management solution succeed here.

I think a good start would be adding klyva to dai-shi's repo again, just to see how well klyva is faring nowadays - and take it from there.

I'll see if I can do it during the weekend :D

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

No branches or pull requests

2 participants