Skip to content
This repository has been archived by the owner on Nov 19, 2024. It is now read-only.

[GraphQl] Fixed response for removeCouponFromCart mutation #4318

Closed

Conversation

rogyar
Copy link
Contributor

@rogyar rogyar commented Apr 23, 2019

Purpose of this PR

This PR introduces a fix for response of removeCouponFromCart mutation documented in https://devdocs.magento.com/guides/v2.3/graphql/reference/quote.html. After coupon is removed from the shopping cart, the response contains no coupon code (currently the response contains the same coupon code that was previously applied).

Affected URLs

https://devdocs.magento.com/guides/v2.3/graphql/reference/quote.html

Links to source code

https://github.com/magento/magento2/blob/1d1c2ea4381bcda5b1baec7360dfc33b0b6569c9/app/code/Magento/QuoteGraphQl/etc/schema.graphqls#L267

The removeCouponFromCart mutation returns Cart type which includes applied_coupon type. If no coupon is applied to the shopping cart the applied_coupon field of the Cart object is null

@devops-devdocs
Copy link
Collaborator

An admin must run tests on this PR before it can be merged.

@osrecio osrecio self-assigned this Apr 23, 2019
@keharper keharper added xx2.3.2 Magento 2.3.2 changes Technical Updates to the code or processes that alter the technical content of the doc labels Apr 23, 2019
@keharper keharper self-assigned this Apr 23, 2019
@keharper
Copy link
Contributor

Fixed under #4502, so this is a duplicate now.

@keharper keharper closed this May 14, 2019
@ghost
Copy link

ghost commented May 14, 2019

Hi @rogyar, 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.
Labels
Partner: Atwix partners-contribution PR created by Magento partner Technical Updates to the code or processes that alter the technical content of the doc xx2.3.2 Magento 2.3.2 changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants