Skip to content
This repository was archived by the owner on Dec 19, 2019. It is now read-only.

GraphQL 296 set billing address #321

Merged
merged 6 commits into from
Feb 12, 2019

Conversation

VitaliyBoyko
Copy link
Contributor

@VitaliyBoyko VitaliyBoyko commented Jan 23, 2019

This PR covers setting billing address on the cart.

Fixed Issues (if relevant)

  1. [Cart Operations] Set Billing Address #296: [Cart Operations] Set Billing Address

Example query:

mutation {
 setBillingAddressOnCart(
   input: {
     cart_id: "maskedQuoteId"
     billing_address: {
       address: {
         firstname: "test firstname"
         lastname: "test lastname"
         company: "test company"
         street: ["test street 1", "test street 2"]
         city: "test city"
         region: "test region"
         postcode: "887766"
         country_code: "US"
         telephone: "88776655"
         save_in_address_book: false
       }
       use_for_shipping: true
     }
   }
 )
}

PS: PR based on my previous work (implementing multi-shipping).
In order to see changes properly, I've done PR to the multi-shipping branch.
I gonna change the base branch since multi-shipping be merged.

@VitaliyBoyko VitaliyBoyko requested a review from naydav January 23, 2019 14:29
@VitaliyBoyko VitaliyBoyko changed the title GraphQA 296 set billing address GraphQL 296 set billing address Jan 23, 2019
@magento-cicd2
Copy link
Contributor

magento-cicd2 commented Feb 7, 2019

CLA assistant check
All committers have signed the CLA.

@VitaliyBoyko VitaliyBoyko changed the base branch from graph-ql-multishipping-coverage to 2.3-develop February 7, 2019 17:25
@VitaliyBoyko VitaliyBoyko force-pushed the GraphQl-296-set-billing-address branch from 1320e52 to 548e33e Compare February 7, 2019 18:45
@ghost
Copy link

ghost commented Feb 12, 2019

Hi @VitaliyBoyko, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.

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

Successfully merging this pull request may close these issues.

4 participants