bigcommerce / catalyst

Catalyst - for Composable Commerce
https://catalyst-demo.site
MIT License
117 stars 150 forks source link

refactor(core): add Suspense boundaries & loading state for Orders #1680

Open bc-alexsaiannyi opened 1 week ago

bc-alexsaiannyi commented 1 week ago

What/Why?

This is 2nd PR for refactoring Orders. It updates <ProductSnippet />, adds Suspense boundaries and Skeleton for Order item while data is being loading.

Testing

Locally

https://github.com/user-attachments/assets/06ea86c9-a835-445b-88e5-2140afa886ed

vercel[bot] commented 1 week ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
catalyst-latest ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 26, 2024 10:23am
catalyst-soul ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 26, 2024 10:23am
3 Skipped Deployments | Name | Status | Preview | Comments | Updated (UTC) | | :--- | :----- | :------ | :------- | :------ | | **catalyst** | ⬜️ Ignored ([Inspect](https://vercel.com/bigcommerce-platform/catalyst/DWNFGaQFwnpiXwHXZ2xchbFfem7d)) | | | Nov 26, 2024 10:23am | | **catalyst-au** | ⬜️ Ignored ([Inspect](https://vercel.com/bigcommerce-platform/catalyst-au/8Ree7uVDs5iVacWjED2zR7ZodXpN)) | [Visit Preview](https://catalyst-au-git-update-product-snip-b1b1cc-bigcommerce-platform.vercel.app) | | Nov 26, 2024 10:23am | | **catalyst-uk** | ⬜️ Ignored ([Inspect](https://vercel.com/bigcommerce-platform/catalyst-uk/9FcV6YiwxPFCzFRYRJPUcmpE5gcD)) | [Visit Preview](https://catalyst-uk-git-update-product-snip-e2667e-bigcommerce-platform.vercel.app) | | Nov 26, 2024 10:23am |
changeset-bot[bot] commented 1 week ago

⚠️ No Changeset found

Latest commit: e9ff0f184d02c90710802ed2bffb3c38c7062020

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

github-actions[bot] commented 4 days ago

⚡️🏠 Lighthouse report

Lighthouse ran against https://catalyst-latest-b2mp5e3a5-bigcommerce-platform.vercel.app

🖥️ Desktop

We ran Lighthouse against the changes on a desktop and produced this report. Here's the summary:

Category Score
🟠 Performance 81
🟢 Accessibility 96
🟢 Best practices 100
🟠 SEO 82

📱 Mobile

We ran Lighthouse against the changes on a mobile and produced this report. Here's the summary:

Category Score
🟢 Performance 94
🟢 Accessibility 96
🟢 Best practices 100
🟠 SEO 85