-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
[Block Editor]: Register the store through registerStore
#47488
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The revert is good, are the test changes related?
Yes. They were copied from Adam's PR. He removed them when the |
Size Change: +6 B (0%) Total Size: 1.31 MB
ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @ntsekouras! The unit tests look good as well – thank you for remembering about them!
Flaky tests detected in 395639a. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/4023576621
|
What?
Reverts: #47447
Why?
See details here: #47447 (comment)
We will be able to use the
register
function once we switch the "preferences" persistence to use the new preferences package.How?
Testing Instructions
registerStore
Most used blocks
are preserved on reload.