magento / graphql-ce

[ARCHIVED] Please use magento/magento2 project
https://github.com/magento/magento2
Open Software License 3.0
131 stars 156 forks source link

Add Merge Cart Mutation #950

Closed pmclain closed 5 years ago

pmclain commented 5 years ago

Description (*)

Add mutation for merging carts.

Guest Use Cases:

Customer Use Cases:

Fixed Issues

  1. magento/graphql-ce#905

Manual testing scenarios (*)

  1. Create two carts
  2. Add products to each cart
  3. mutation {
    mergeCarts(input: {
    first_cart_id: "{$firstMaskedId}"
    second_cart_id: "{$secondMaskedId}"
    })
    }

Expected Result Cart contents are merged. A new masked cart id is returned and the previous masked ids return a not found error.

Contribution checklist (*)

lenaorobei commented 5 years ago

Need to move this PR to on hold until we will finalize product requirements with @nrkapoor https://github.com/magento/architecture/pull/288

lenaorobei commented 5 years ago

Sorry to say that but I need to close this PR due to requirement changes. The new flow we'll be working on:

Screen Shot 2019-10-14 at 9 25 11 AM
ghost commented 5 years ago

Hi @pmclain, 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.

ghost commented 5 years ago

Hi @pmclain, 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.