-
Notifications
You must be signed in to change notification settings - Fork 172
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
refactor(store): move old redux code to redux toolkit #594
Conversation
Hi @sekwah41 this PR is ready for now. The new redux code for the tasks store is missing, but we can migrate it in other PR (this PR is big enought and the tasks store is more complex as it uses redux-undo). |
Ill try to review this tomorrow, though will review it at least at some point next week :) |
Hey @sekwah41 any news on this? I know it's a big PR but I think it's worth it having this merged before creating another store to save the custom shortcuts settings |
Sorry my bad. I should have time tomorrow but if I can I'll do it tonight. |
Tested on both the electron and tauri versions and seems fine |
As the official docs says the
createStore
method is deprecated. This PR moves all the stores to redux-toolkit which is: