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.

Sign up for GitHub

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

Jump to bottom

INT-1019: Sync guest carts for checkout #2

Merged
merged 3 commits into from
Jun 22, 2022
Merged

INT-1019: Sync guest carts for checkout #2

merged 3 commits into from
Jun 22, 2022

Conversation

hugocore
Copy link
Contributor

@hugocore hugocore commented Jun 22, 2022 •
edited
Loading

Acceptance Criteria

  • Given a guest user without an expiring Access Token
  • and a cart with multiple products
  • When redirecting to the Checkout Sync controller
  • and passing the Cart ID
  • Then the guest cart is synced to checkout
  • and the user is redirected to the checkout page

Dev

  • Extension exposes a GET endpoint that optionally accepts token and requires a cartId, e.g.
https://magento.test/frontendcheckout/cart/sync?cartId=vhEjyHgGyB0z76pTMpzRZZtZzCJ5v49U
  • The guest cart ID will be masked so first it needs to be decrypted into an internal quote ID
  • The extension then loads the internal quote behind the masked Cart ID and syncs for checkout
  • The user is then redirected to the checkout page

Demo

Screen.Recording.2022-06-22.at.12.28.14.mov

@hugocore hugocore self-assigned this Jun 22, 2022
Draiken
Draiken approved these changes Jun 22, 2022
View reviewed changes
Service/CheckoutSync.php Outdated Show resolved Hide resolved
@hugocore hugocore merged commit 7117fcb into main Jun 22, 2022
@hugocore hugocore deleted the INT-1019-sync-guest-cart branch June 22, 2022 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Draiken Draiken Draiken approved these changes

@Andy-H-Nguyen Andy-H-Nguyen Andy-H-Nguyen approved these changes

@rodrigoi rodrigoi Awaiting requested review from rodrigoi

Assignees

@hugocore hugocore

Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hugocore @Draiken @Andy-H-Nguyen