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

GraphQL: Add support for downloadable/gift card products in customer … #7716

Merged
merged 9 commits into from
Aug 17, 2020

Conversation

keharper
Copy link
Contributor

@keharper keharper commented Aug 11, 2020

…orders

Purpose of this pull request

This pull request (PR) adds attributes that support downloadable products and gift cards in a customer order.

I also added schema changes made in internal ticket MC-35987

Affected DevDocs pages

  • ...

Links to Magento source code

  • ...

@keharper keharper added Internal Dev Differentiates work between community and Magento staff Technical Updates to the code or processes that alter the technical content of the doc 2.4.1 labels Aug 11, 2020
@keharper keharper self-assigned this Aug 11, 2020
@keharper keharper marked this pull request as draft August 11, 2020 19:33
@keharper keharper marked this pull request as ready for review August 12, 2020 19:46
@keharper keharper requested a review from erikmarr August 17, 2020 20:16
Copy link
Contributor

@erikmarr erikmarr left a comment

Choose a reason for hiding this comment

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

Looks good...just one minor question.

--- | --- | ---
`additional_data` | [[KeyValue](#KeyValue)] | Additional data per payment method type
`name` | String! | The label that describes the payment method
`type` | String! | The payment method code that indicates how the order was paid for
Copy link
Contributor

Choose a reason for hiding this comment

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

can we/should we list the possible payment method codes that this attribute can return?

Copy link
Contributor Author

@keharper keharper Aug 17, 2020

Choose a reason for hiding this comment

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

No. The payment reference section is meant to be definitive source of that. Moreover, 3rd party payment processors should be supporting GraphQL now or soon, and we can't know their values.

@keharper
Copy link
Contributor Author

running tests

@keharper keharper merged commit 151c05f into 2.4.1-develop Aug 17, 2020
@ghost
Copy link

ghost commented Aug 17, 2020

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

@keharper keharper deleted the kh_downloadable branch August 19, 2020 19:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
2.4.1 Internal Dev Differentiates work between community and Magento staff Technical Updates to the code or processes that alter the technical content of the doc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants