Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Refactor checkout workflow to set shipping address/get shipping methods with graphql #2018

Merged
merged 6 commits into from
Dec 18, 2019

Conversation

sirugh
Copy link
Contributor

@sirugh sirugh commented Dec 6, 2019

Description

Uses graphQL to set shipping address and, in the same mutation, receive the available shipping methods for the selected address.

Also removes email from forms for authed users as the email is already attached to cart.
For guest users it will set the guest email on the cart when submitting the address form.

Related Issue

Closes PWA-145.

Acceptance

Verification Stakeholders

Specification

Verification Steps

  1. Go through checkout -- it should complete successfully. Verify that the REST submissions still work.

Screenshots / Screen Captures (if appropriate)

Checklist

  • I have updated the documentation accordingly, if necessary.
  • I have added tests to cover my changes, if necessary.

@PWAStudioBot
Copy link
Contributor

PWAStudioBot commented Dec 6, 2019

Messages
📖

Access a deployed version of this PR here. Make sure to wait for the "pwa-pull-request-deploy" job to complete.

📖 DangerCI Failures related to missing labels/description/linked issues/etc will persist until the next push or next nightly build run (assuming they are fixed).
📖

Associated JIRA tickets: PWA-145.

Generated by 🚫 dangerJS against e35dc57

@sirugh sirugh force-pushed the rugh/getShippingMethods-graphql branch 2 times, most recently from bdf3137 to 4030d43 Compare December 12, 2019 19:20
@sirugh sirugh added the version: Major This changeset includes incompatible API changes and its release necessitates a Major version bump. label Dec 12, 2019
@sirugh sirugh force-pushed the rugh/getShippingMethods-graphql branch from 4030d43 to e89c9be Compare December 12, 2019 19:27
@sirugh sirugh marked this pull request as ready for review December 12, 2019 19:54
Copy link
Contributor

@supernova-at supernova-at left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just the one error message regression - great job!


const [
{ shippingAddress, shippingMethod, shippingTitle }
] = useCheckoutContext();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯

supernova-at
supernova-at previously approved these changes Dec 16, 2019
@dpatil-magento
Copy link
Contributor

QA pass but waiting for #2029 to get merged first.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
pkg:peregrine pkg:venia-ui version: Major This changeset includes incompatible API changes and its release necessitates a Major version bump.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants