Skip to content

Conversation

@jhadobe
Copy link
Contributor

@jhadobe jhadobe commented Sep 5, 2025

In conjunction with AdobeDocs/commerce-extensibility#444, this PR removes spaces from SKUs in example code, since they are causing issues in the integration starter kit.

@jhadobe jhadobe self-assigned this Sep 5, 2025
@jhadobe jhadobe added editorial Typo and grammar fixes or minor rewrites to correct inaccuracies feedback labels Sep 5, 2025
@github-project-automation github-project-automation bot moved this to 📋 Needs Review in Commerce - Pull Requests Sep 5, 2025
@jhadobe jhadobe marked this pull request as ready for review September 5, 2025 20:25
@meker12 meker12 requested a review from Copilot September 5, 2025 20:49
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes spaces from SKU values in example code to comply with Commerce SKU requirements, which disallow spaces in SKU identifiers.

  • Updates SKU format from space-separated to hyphen-separated (e.g., "Simple Product 1" → "Simple-Product-1")
  • Ensures consistency across REST API import examples and GraphQL B2B purchase order examples
  • Aligns documentation with Commerce platform constraints

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
src/pages/rest/modules/import/index.md Updates SKU values in REST import API examples to use hyphens instead of spaces
src/pages/graphql/schema/b2b/purchase-order/mutations/add-items-to-cart.md Updates SKU value in GraphQL mutation example to use hyphens
src/pages/graphql/schema/b2b/purchase-order/index.md Updates SKU value in GraphQL query example to use hyphens

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-project-automation github-project-automation bot moved this from 📋 Needs Review to 👍 Approved in Commerce - Pull Requests Sep 5, 2025
@jhadobe jhadobe merged commit 3e55a88 into main Sep 5, 2025
32 checks passed
@github-project-automation github-project-automation bot moved this from 👍 Approved to 🏁 Done in Commerce - Pull Requests Sep 5, 2025
@jhadobe jhadobe deleted the jh_fix-sku branch September 5, 2025 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

editorial Typo and grammar fixes or minor rewrites to correct inaccuracies feedback

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants