-
Notifications
You must be signed in to change notification settings - Fork 179
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
Assign cart to customer as part of login mutation #1765
Conversation
🦋 Changeset detectedLatest commit: 3108753 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
3 Skipped Deployments
|
72e39e2
to
2643cf5
Compare
2643cf5
to
a2f9cb1
Compare
f000add
to
b0ce62a
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.
nice
b0ce62a
to
3108753
Compare
⚡️🏠 Lighthouse reportLighthouse ran against https://catalyst-latest-bqmt11nkf-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:
|
What/Why?
Use the new enhancement to the
login()
mutation to assign the guest cart to the customer at the moment of login, which avoids an extra GQL request for this and simplifies the code.A similar enhancement is coming soon to logout but will be a separate PR.
Testing
Preview deployment
Screen.Recording.2024-12-16.at.4.22.45.PM.mov