-
Notifications
You must be signed in to change notification settings - Fork 147
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
The Lesser Migration #2470
The Lesser Migration #2470
Conversation
Passing run #3913 ↗︎Details:
This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. |
Performance Test Results:
|
.dev/tests/phpunit/includes/block-migrate/test-coblocks-block-migration.php
Outdated
Show resolved
Hide resolved
This reverts commit c43889e.
At least the migration from alert to paragraph does not work well with UTF-8 characters (e.g., Umlaute as often used on German sites, sucha as ä, ö or ü) and is currently destroying one page after another on my sites, whenever I want to change anything else on the page, leading to much extra work of changing the butchered characters back. The list of changes is unfortunately very extensive in this PR, making it hard for me to find an entry point to submit a fix – could you kindly give an indication on where to start, dear @AnthonyLedesma ? |
Hi @GenieTim, I am so sorry to hear about that fact with UTF8 chars. This fix I think has to do with how the content is loaded into PHP and parsed locally. It seems like the content is being imported in a way that does not handle some characters. I took a quick look and tested it out with special characters, made some modifications to the migrate logic, and have a PR that is close to ready. I hope to have this fix out soon. |
PR up to resolve special characters bug. |
Description
The Great Migration POC demonstrated a successful approach for migrating CoBlocks blocks into Core blocks, providing users with a more consistent and unified experience. The Lesser Migration builds on this success by implementing a staged approach, allowing us to gradually introduce this experience starting with a smaller audience.
As the Gutenberg landscape has evolved, it has become clear that Core blocks offer quality user experience, consistent controls, and regular updates. While it is possible to align CoBlocks with Core, it has proven to be a challenging task to back-port features and functionalities. Additionally, migrations to new features and functionality often require deprecating blocks, adding to bloat and load times.
In summary, The Lesser Migration aims to streamline the user experience by migrating select blocks to Core, reducing maintenance and update overhead, and allowing us to focus on unique and user-enhancing features in CoBlocks.
Also requires a Go branch for proper Go-specific styling.
godaddy-wordpress/go#838
Screenshots
Alert Block
![migrateAlertBlock](https://user-images.githubusercontent.com/30462574/215218208-02c9d293-4190-460a-b931-a061c6df53e8.gif)
Author Block
![migrateAuthorBlock](https://user-images.githubusercontent.com/30462574/215218264-3953bd62-6779-4cab-9bbb-f3d3aff52f7e.gif)
Types of changes
Major changes intended for CoBlocks 3.0 release.
PHP and JS changes introducing block migrations.
How has this been tested?
Manually tested minor CoBlocks versions back to 2.20.0 all able to successfully migrate.
PHPUnit tests in place. Unit tests cover:
To test this manually:
First, you must set up a 'master' branch of CoBlocks locally. You want to create two new posts. One post is to test the Author block and another is to test the Alert block. When running the block editor and 'master' branch you should insert blocks into the post. We should use features of the block as well. For example, we should have a block with custom colors. A block with custom alignment. Block with a custom class name. A Block using any and all features such as block animation or typography controls like bold and italics.
Once we have a test post for each block, and each test post is filled with configured blocks, we are ready to test!
Next, we need to check out this branch
cb-30-base
and build. After building these changes, the next time we load one of the test posts, it will migrate the CoBlocks blocks into Core blocks.For reference:
Alert Block -> Paragraph styled as an alert.
Author Block -> Columns block layout as author.
Acceptance criteria
Checklist: