-
Notifications
You must be signed in to change notification settings - Fork 199
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
chore: upgrade @makeswift/runtime
to 0.23.1
#1929
base: integrations/makeswift
Are you sure you want to change the base?
chore: upgrade @makeswift/runtime
to 0.23.1
#1929
Conversation
This commits also updates: - Make the Shape prop required. Makeswift supports this by default. - Add default values to height and weight that matches the Soul default values. - Move the banner id prop to be after allowClose. - Rename and move some variables.
This was causing HTML hydration error when we put button inside carousel: In HTML, <a> cannot be a descendant of <a>. We also don't want the carousel to be inside a link.
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
4 Skipped Deployments
|
a0b7f26
to
6fa9baa
Compare
⚡️🏠 Lighthouse reportLighthouse ran against https://catalyst-canary-2m9nq9vii-bigcommerce-platform.vercel.app 🖥️ DesktopWe ran Lighthouse against the changes on a desktop and produced this report. Here's the summary:
📱 MobileWe ran Lighthouse against the changes on a mobile and produced this report. Here's the summary:
|
856b125
to
a38aaff
Compare
What/Why?
Upgrade
@makeswift/runtime
to the latest official release.Testing
Kapture.2025-01-23.at.22.00.25.mp4