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

Conversation

karyna-t
Copy link
Contributor

@karyna-t karyna-t commented Nov 16, 2021

Purpose of this pull request

This pull request (PR) adds a documentation page for the assignCustomerToGuestCart mutation (GraphQL).

Magento 2 GitHub Issue: magento/magento2#32615
Magento 2 GitHub PR: magento/magento2#33106

Fixes #9248

Affected DevDocs pages

  • ...

Links to Magento source code

  • ...

whatsnew
Added the assignCustomerToGuestCart mutation.

Copy link
Contributor

@andrewbess andrewbess left a comment

Choose a reason for hiding this comment

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

Hello @karyna-tsymbal-atwix
Thank you for your contribution.
It looks good from technical side.

@dobooth
Copy link
Contributor

dobooth commented Nov 17, 2021

@keharper

@keharper keharper self-requested a review November 18, 2021 21:13
@keharper keharper added 2.4.4 Community Docs impacted by community code contribution New Topic A major update published as an entirely new document Waiting for Response Waiting for response from internal/external parties labels Nov 27, 2021
@karyna-t karyna-t requested a review from keharper December 1, 2021 16:13

The `assignCustomerToGuestCart` mutation assigns a logged in customer and merge customer's shopping cart items to the specified guest shopping cart.

The customer must be logged in (You must specify the customer’s authorization token in the headers).
Copy link
Contributor

Choose a reason for hiding this comment

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

Replace this line with the standard "This mutation requires a valid customer authentication token." and move it to the bottom of this section.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

Customer now is assigned to that active cart.

*Note:*
Customer cart becomes inactive and the guest cart remains active; but for guest cart new `masked_id` is being generated.
Copy link
Contributor

Choose a reason for hiding this comment

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

Why are masked_id and quote_id mentioned here, when neither are part of the request or response?

Copy link
Contributor Author

@karyna-t karyna-t Dec 12, 2021

Choose a reason for hiding this comment

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

Hi @keharper , I mentioned them because I think it's a useful information for the developers, who can be confused why there is no old masked_id in the database, but a new one appeared instead.
I can remove this note if you think that it's unnecessary information.

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Customer cart becomes inactive and the guest cart remains active; but for guest cart new `masked_id` is being generated.
The `masked_id` of the guest cart contains a new value. The `quote_id` remains the same.

@dobooth
Copy link
Contributor

dobooth commented Dec 10, 2021

@karyna-tsymbal-atwix

@karyna-t karyna-t force-pushed the docs-for-assignCustomerToGuestCart-graphql-mutation branch from 3e09345 to 2db4e9d Compare December 12, 2021 09:45
@karyna-t karyna-t force-pushed the docs-for-assignCustomerToGuestCart-graphql-mutation branch from 2db4e9d to d247290 Compare December 12, 2021 09:46
@dobooth dobooth requested a review from keharper December 14, 2021 16:33
Customer now is assigned to that active cart.

*Note:*
Customer cart becomes inactive and the guest cart remains active; but for guest cart new `masked_id` is being generated.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Customer cart becomes inactive and the guest cart remains active; but for guest cart new `masked_id` is being generated.
The `masked_id` of the guest cart contains a new value. The `quote_id` remains the same.

@keharper
Copy link
Contributor

keharper commented Jan 5, 2022

@magento import code to magento-devdocs/devdocs

@m2-github-services
Copy link

@keharper the branch with the code is successfully imported into the magento-devdocs/devdocs repository. Branch name: imported-karyna-tsymbal-atwix-devdocs-9252.

@keharper keharper changed the base branch from master to 2.4.4-develop January 5, 2022 17:36
@magento-devops-reposync-svc magento-devops-reposync-svc merged commit ca69907 into magento:2.4.4-develop Jan 5, 2022
@hguthrie hguthrie removed the Waiting for Response Waiting for response from internal/external parties label Jun 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
2.4.4 Community Docs impacted by community code contribution New Topic A major update published as an entirely new document Partner: Atwix partners-contribution PR created by Magento partner Progress: done
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GraphQL: Add docs for assignCustomerToGuestCart mutation
7 participants