-
Notifications
You must be signed in to change notification settings - Fork 145
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
build(react18)!: drop support for react v16 and v17 #7666
base: master
Are you sure you want to change the base?
build(react18)!: drop support for react v16 and v17 #7666
Conversation
✅ Deploy Preview for ecommerce-app-base-components ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
015e939
to
c1e5270
Compare
packages/ecommerce-app-base/src/ProductCard/ProductImage/ProductImage.tsx
Show resolved
Hide resolved
packages/ecommerce-app-base/src/Editor/SortableComponent.stories.tsx
Outdated
Show resolved
Hide resolved
skuType, | ||
}) => { | ||
const [productPreviews, setProductPreviews] = useState<Product[]>([]); | ||
const getProductPreviews = useCallback(async () => { |
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.
- todo: see if we can re-implement
shouldRefetch()
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.
done, see usePreviousSkus()
hook
29bbcbe
to
6090fc6
Compare
…rtable-hoc with dnd-kit BREAKING CHANGE: Consumers of ecommerce-app-base are required to implement react v18
6090fc6
to
61327d5
Compare
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.
Awesome! 🚀 Thanks for making the changes we discussed, this LGTM!
Purpose
react v16 >=
react-sortable-hoc
library with new dndkit that supports react v18 <=BREAKING CHANGE: Dropping support for
react-dom <= 17
andreact <= 17
. Consumers of ecommerce-app-base are required to implement react v18.Testing steps
Shopify
Tested locally by
npm pack
'ing aecommerce-app-base
tar file, and installing it locally inside shopify.Also tested with Storybook