-
Notifications
You must be signed in to change notification settings - Fork 0
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
Update for WordPress 5.8 compatibility #4
Comments
The block's now been refactored. Before closing the issue I need to dot the i's and cross the t's... delivering the new version on oik-plugins and updating the documentation. |
Oops. the transform from Verse doesn't work since |
I should also complete the internationalization and localization. |
oik-magnetic-poetry v0.2.2 is now available from oik-plugins.com. |
The oik-magnetic-poetry plugin was originally built using webpack.
When used in WordPress 5.8 the Widget block editor produces a "doing it wrong" message due to the enqueueing of wp-editor.
It seems the best solution is to rework the code to build it using wp-scripts which will enable use of
import ServerSideRender from '@wordpress/server-side-render';
See bobbingwide/bobbingwide#30 (comment)
Requirements
block.json
Proposed solution
webpack.config.js
and.babelrc
package.json
andblock.json
for oik-mp/magnetic-poetryoik-magnetic-poetry/index.js
to register the block fromblock.json
oik-magnetic-poetry/index.js
to add logic for text alignoik-magnetic-poetry.php
to register the block fromblock.json
oik-magnetic-poetry.php
to useget_block_wrapper_attributes()
node_modules
and rebuildProcess for updating node_modules
node_modules
foldernpm install
npm install @wordpress/scripts --save-dev
See https://developer.wordpress.org/block-editor/reference-guides/packages/packages-scripts/
This worked first time! In the next plugin just 1. and 2. or 1. and 3. and see if the build works
The text was updated successfully, but these errors were encountered: