-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Reduxify: Notices #48408
Comments
I'm going to start working on this one. |
#48672 reduxifies the notice in |
#48689 reduxifies notices in SEO settings. |
A few more PRs here:
|
Several more reduxification or cleanup PRs today:
|
Several more PRs today to reduxify the last bits: #49530 - Purchases: Reduxify notices in Furthermore, #49461 takes a stab at typing the notices actions, and #49544 deprecates the old notices library. |
#49600 has been open to reduxify notices in the |
Final PR to remove the legacy notices store once all the above PRs are merged: #49663 🎉 |
All PRs have been merged, only the last one for final removal remains: #49663 |
This
EventEmitter
store is used quite a bit in Calypso (48 occurrences at this time) for managing global notices (source).We already have a working (and quite well adopted) Redux implementation, so to get away from the
EventEmitter
implementation, we have to port every usage to use Redux. Those refactors are usually trivial, but it'll take some time to carry them all out.The text was updated successfully, but these errors were encountered: