-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Update Customer GraphQL Documentation to Include Mutations #3740
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some big-picture things before I start reviewing at the sentence level.
- Create a head2 called Parameters and move all the parameter tables under that.
- Find a way to consolidate the Customer tables. That's too much duplicate info.
- Create subsections under Mutations: Manage customers, manage customer addresses, manage customer tokens
- Your example usage statements are H2s. They should be a lower heading than what's used for the mutation name.
@keharper I have made several changes, but am struggling with some of these suggestions. I feel like we should keep the queries and mutations as separate tables/sections since they cannot be used together in the same graphql call (at least that is my understanding). I propose a structure like: Queries Mutations Here's a new build with the above proposed structure: |
@keharper, I just posted a new build for review: |
closing as this is resolved in the following PR: #3818 |
Hi @erikmarr, thank you for your contribution! |
This PR is a:
Summary
When this pull request is merged, it will fix issue #3338.
This PR updates the graphql documentation to include the new mutations.
Additional information
List all affected URLs