-
Notifications
You must be signed in to change notification settings - Fork 219
Replace react imports with wordpress/element #8520
Conversation
The release ZIP for this PR is accessible via:
Script Dependencies ReportThe
This comment was automatically generated by the TypeScript Errors Report
assets/js/base/components/cart-checkout/totals/coupon/index.tsx
assets/js/base/components/read-more/index.tsx |
Size Change: -2.27 kB (0%) Total Size: 1.1 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.
Massive work here; well done 🎉 ! I left a couple of questions, but pre-approving.
Run npm run lint:js and verify there are no errors (there are some warnings, but they are unrelated).
Can you confirm what the expected errors are? Are these the warnings that are a-ok?
Run several scripts and verify they still work (npm run start, npm run storybook and npm run build).
✅ Confirmed all these scripts are working as expected.
Do some smoking testing with the browser console open and verify there are no error logs related to the changes in this PR: try adding blocks in the editor, viewing them in the frontend, interacting with some of the options...
✅ I didn't test all blocks, but a handful (filter blocks, a few atomic blocks, mini-cart, checkout), and everything works as expected. Should we be more detail-oriented here and review them all one by one, or are we a-ok with a superficial test?
Thanks for the review @nefeline!
I think we should still fix or investigate them, but since they are reproducible in Btw, I pushed a couple more fixes in 1068dae, which were introduced these last days. |
Agreed! |
👍 I'm taking another look here quickly to confirm things are a-ok after the latest commit. |
All good! 🙌 |
Fixes #6732.
This PR:
react
with imports from@wordpress/element
except for type imports.Testing
User Facing Testing
npm run lint:js
and verify there are no errors (there are some warnings, but they are unrelated).npm run start
,npm run storybook
andnpm run build
).