Skip to content

Conversation

@RakeshJesadiya
Copy link
Contributor

Problem

Customer Reorder Schema needs to update id with UID changes.

Solution

With consistency throughout the graphql schema, we have replaced the Id field with Uid.

Requested Reviewers

@RakeshJesadiya
Copy link
Contributor Author

@cpartica Please check the PR and give me your valuable input. Thanks.

@@ -1,5 +1,5 @@
type Mutation {
reorderItems(orderId: ID!): ReorderItemsOutput! @doc(description: "Add all items from the registered customer's order to his shopping cart.")
Copy link
Contributor

Choose a reason for hiding this comment

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

reorderItems(orderId: ID, orderId: ID) - we will accept both and enforce this through code not schema

Copy link
Contributor

Choose a reason for hiding this comment

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

hi @cpartica, please confirm that this will be implemented and please close this ticket

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants