Skip to content

DATA-7320: Document the unified product media gallery - #627

Merged
erikmarr merged 4 commits into
AdobeDocs:mainfrom
magento-slow-horses:DATA-7320-product-media-gallery
Oct 20, 2026
Merged

erikmarr merged 4 commits into
AdobeDocs:mainfrom
magento-slow-horses:DATA-7320-product-media-gallery

Conversation

@slavvka

@slavvka slavvka commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

Purpose of this pull request

Document the Catalog Service gallery field and ProductViewMedia union. The gallery returns product images and videos in resolved display order, which clients should preserve.

Affected pages

…ort + unified gallery field

 - Document the unified product media gallery GraphQL field

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@slavvka
slavvka marked this pull request as ready for review October 7, 2026 17:23
@erikmarr erikmarr self-assigned this Oct 8, 2026
@erikmarr erikmarr added the technical Changes to code, examples, or technical processes label Oct 8, 2026
@erikmarr

erikmarr commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

Thanks for your contribution @slavvka! Does this change also affect the merchandising services API? https://developer.adobe.com/commerce/services/optimizer/

@slavvka

slavvka commented Oct 13, 2026

Copy link
Copy Markdown
Contributor Author

@erikmarr Yes, gallery also affects the Merchandising GraphQL API. I opened commerce-services PR 358 to document it.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The documentation updates consistently match the stated scope, with no blocking issues identified.

0 open findings

What changed in this PR

Documents the Catalog Service unified media gallery across the shared ProductView reference and three query pages.

Changes:

  • Adds gallery to the ProductView interface and concrete type tables.
  • Explains ProductViewMedia inline fragments and preserving returned display order.
File Description
src/​pages/​includes/​graphql/​catalog-service/​product-view.md Updates the shared media gallery reference.
src/​pages/​graphql/​schema/​catalog-service/​queries/​refine-product.md Documents gallery support for product refinement.
src/​pages/​graphql/​schema/​catalog-service/​queries/​products.md Documents gallery support for product queries.
src/​pages/​graphql/​schema/​catalog-service/​queries/​product-variants.md Documents gallery support for variant queries.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@github-project-automation github-project-automation Bot moved this from 📋 Needs Review to 👍 Approved in Commerce - Pull Requests Oct 15, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🔍 Linter Report

Click to expand full report
═══════════════════════════════════════════════════════════════
                     LINTER REPORT
═══════════════════════════════════════════════════════════════

Generated: 2026-10-15T14:33:49.655Z
Mode: Full Linting (all rules + dead links check)
Target Directory: /home/runner/work/commerce-webapi/commerce-webapi
Skipped Rules:
  - check-frontmatter → src/pages/includes
  - missing-heading → src/pages/includes

───────────────────────────────────────────────────────────────

Files to process: 4


═══════════════════════════════════════════════════════════════
                        SUMMARY
═══════════════════════════════════════════════════════════════

  📁 Files processed:    4
  📄 Files with issues:  0
  ❌ Total errors:       0
  ⚠️  Total warnings:     0
  📋 Total issues:       0

Result: ✅ PASSED - All files passed linting successfully!

═══════════════════════════════════════════════════════════════

This comment was automatically generated by the linter bot.

@erikmarr
erikmarr merged commit 6d730b8 into AdobeDocs:main Oct 20, 2026
6 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

🔍 Linter Report

Click to expand full report
═══════════════════════════════════════════════════════════════
                     LINTER REPORT
═══════════════════════════════════════════════════════════════

Generated: 2026-10-20T20:11:55.113Z
Mode: Full Linting (all rules + dead links check)
Target Directory: /home/runner/work/commerce-webapi/commerce-webapi
Skipped Rules:
  - check-frontmatter → src/pages/includes
  - missing-heading → src/pages/includes

───────────────────────────────────────────────────────────────

Files to process: 4


═══════════════════════════════════════════════════════════════
                        SUMMARY
═══════════════════════════════════════════════════════════════

  📁 Files processed:    4
  📄 Files with issues:  0
  ❌ Total errors:       0
  ⚠️  Total warnings:     0
  📋 Total issues:       0

Result: ✅ PASSED - All files passed linting successfully!

═══════════════════════════════════════════════════════════════

This comment was automatically generated by the linter bot.

@github-project-automation github-project-automation Bot moved this from 👍 Approved to 🏁 Done in Commerce - Pull Requests Oct 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

technical Changes to code, examples, or technical processes

Projects

Status: 🏁 Done

Development

Successfully merging this pull request may close these issues.

4 participants