Skip to content
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

(REFERENCE) Add gift certificates page with purchase, check, and redeem tabs #1486

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

becomevocal
Copy link
Contributor

What/Why?

Adds a gift certificate page with purchase, check, and redeem tab similar to what Cornerstone has.

Uses our REST API for checking the certificate balance, which requires an additional GIFT_CERTIFICATE_V3_API_TOKEN env var. Suggest creating a read-only token if you go this route. Adding a gift certificate to a cart uses GraphQL.

The route is at /gift-certificates.

Here's a demo:

gift-certificates-on-catalyst-720p.mov

Under the hood it supports localization using our existing standards within Catalyst.

Backstory

I've been looking for something tangible to test v0.dev with and this fit the bill! It ended up doing a pretty good job out of the box, with my first prompt that asked for a gift certificate balance checker using the BigCommerce API.

Initial v0 prompt and result

Screenshot 2024-10-21 at 2 22 17 AM

Ended up having to switch out the components since it defaults to shadcn components, fix the Gift Certificate API request that was using an invalid route, and manually polish areas to follow our standards. Like using our GraphQL client.

Overall it's a good pairing experience, especially when thinking through app router / RSCs.

Testing

🎥 See video ^

@becomevocal becomevocal requested a review from a team as a code owner October 21, 2024 06:43
Copy link

changeset-bot bot commented Oct 21, 2024

⚠️ No Changeset found

Latest commit: 290ccf2

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

Copy link

vercel bot commented Oct 21, 2024

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 Oct 31, 2024 6:11am
6 Skipped Deployments
Name Status Preview Comments Updated (UTC)
catalyst ⬜️ Ignored (Inspect) Oct 31, 2024 6:11am
catalyst-1millionproducts-store ⬜️ Ignored (Inspect) Visit Preview Oct 31, 2024 6:11am
catalyst-au ⬜️ Ignored (Inspect) Visit Preview Oct 31, 2024 6:11am
catalyst-test-store ⬜️ Ignored (Inspect) Visit Preview Oct 31, 2024 6:11am
catalyst-uk ⬜️ Ignored (Inspect) Visit Preview Oct 31, 2024 6:11am
catalyst-unstable ⬜️ Ignored (Inspect) Visit Preview Oct 31, 2024 6:11am

Copy link
Contributor

⚡️🏠 Lighthouse report

Lighthouse ran against https://catalyst-latest-92hclutb9-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 93
🟢 Accessibility 96
🟢 Best practices 100
🟠 SEO 85

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant