Adding new tutorial to commerce-services by erikmarr · Pull Request #347 · AdobeDocs/commerce-services

erikmarr · 2026-06-23T17:03:51Z

Purpose of this pull request

This pull request adds documentation for implementing checkout analytics on Luma-based Adobe Commerce storefronts. It introduces a new implementation guide that shows developers how to publish the placeOrder storefront event on the checkout success page when cart data is cleared before the page renders.

Affected pages

whatsnew
Added Instrument checkout analytics for a Luma storefront — a step-by-step guide for publishing the placeOrder event on checkout success when cart data is unavailable after order placement.

github-actions · 2026-06-23T17:05:40Z

🔍 Linter Report

Click to expand full report

═══════════════════════════════════════════════════════════════
                     LINTER REPORT
═══════════════════════════════════════════════════════════════

Generated: 2026-06-23T17:05:12.236Z
Mode: Full Linting (all rules + dead links check)
Target Directory: /home/runner/work/commerce-services/commerce-services
Skipped Rules:
  - check-frontmatter → src/pages/includes

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

Files to process: 5

───────────────────────────────────────────────────────────────
📄 FILE: src/pages/config.md
───────────────────────────────────────────────────────────────
  ⚠️  WARNING
    Location: Line 1:1
    Message: Missing frontmatter section - add --- at the beginning with title and description
    Rule: check-frontmatter

  ⚠️  WARNING
    Location: Line 5:7
    Message: Unexpected redirecting URL `https://developer.adobe.com/commerce/docs`, expected final URL `https://developer.adobe.com/commerce/docs/`
    Rule: no-dead-urls

───────────────────────────────────────────────────────────────
📄 FILE: src/pages/shared-services/implementation/tutorial.md
───────────────────────────────────────────────────────────────
  ⚠️  WARNING
    Location: Line 2:1
    Message: Title is too long (71 characters). Consider keeping it under 60 characters.
    Rule: check-frontmatter

  ❌ ERROR
    Location: Line 48:40
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 60:30
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 76:32
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 91:36
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 115:41
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 150:46
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 197:45
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 223:34
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 242:29
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 259:31
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ⚠️  WARNING
    Location: Line 317:3
    Message: Unexpected dead URL `https://developer.adobe.com/commerce/frontend-core/guide/themes/js-mixins/`, expected live URL
    Rule: no-dead-urls

───────────────────────────────────────────────────────────────
📄 FILE: src/pages/shared-services/storefront-events/index.md
───────────────────────────────────────────────────────────────
  ⚠️  WARNING
    Location: Line 104:62
    Message: Unexpected dead URL `https://experienceleague.adobe.com/en/docs/commerce/live-search/live-search-admin/rules/rules-add#intelligent-ranking`, expected live URL
    Rule: no-dead-urls


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

  📁 Files processed:    5
  📄 Files with issues:  3
  ❌ Total errors:       10
  ⚠️  Total warnings:     5
  📋 Total issues:       15

Result: ❌ FAILED - Fatal errors found

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

This comment was automatically generated by the linter bot.

Copilot

Pull request overview

Adds new Shared Services documentation that walks Adobe Commerce (Luma/RequireJS) developers through instrumenting checkout analytics by publishing the placeOrder storefront event on the checkout success page, plus navigation/linking so the tutorial is discoverable from the Storefront Events docs and site nav.

Changes:

Introduces a new step-by-step Luma tutorial for publishing placeOrder on checkout success when cart data is cleared before render.
Adds an “implementation guides” landing page and wires it into Shared Services pages/navigation.
Cross-links the new tutorial from the Storefront Events overview and the docs nav config.

Reviewed changes

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

Show a summary per file

File	Description
`src/pages/shared-services/storefront-events/index.md`	Adds a link from Storefront Events overview to the new Luma checkout analytics tutorial.
`src/pages/shared-services/index.md`	Adds a pointer from Shared Services landing page to the new implementation guides section.
`src/pages/shared-services/implementation/tutorial.md`	New tutorial with module structure and code samples to snapshot cart data and publish `placeOrder` on success.
`src/pages/shared-services/implementation/index.md`	New landing page that lists storefront implementation guides.
`src/pages/config.md`	Adds the new implementation guide pages to the documentation navigation tree.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

github-actions · 2026-06-23T18:11:35Z

🔍 Linter Report

Click to expand full report

═══════════════════════════════════════════════════════════════
                     LINTER REPORT
═══════════════════════════════════════════════════════════════

Generated: 2026-06-23T18:11:00.571Z
Mode: Full Linting (all rules + dead links check)
Target Directory: /home/runner/work/commerce-services/commerce-services
Skipped Rules:
  - check-frontmatter → src/pages/includes

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

Files to process: 5

───────────────────────────────────────────────────────────────
📄 FILE: src/pages/config.md
───────────────────────────────────────────────────────────────
  ⚠️  WARNING
    Location: Line 1:1
    Message: Missing frontmatter section - add --- at the beginning with title and description
    Rule: check-frontmatter

  ⚠️  WARNING
    Location: Line 5:7
    Message: Unexpected redirecting URL `https://developer.adobe.com/commerce/docs`, expected final URL `https://developer.adobe.com/commerce/docs/`
    Rule: no-dead-urls

───────────────────────────────────────────────────────────────
📄 FILE: src/pages/shared-services/implementation/tutorial.md
───────────────────────────────────────────────────────────────
  ⚠️  WARNING
    Location: Line 2:1
    Message: Title is too long (71 characters). Consider keeping it under 60 characters.
    Rule: check-frontmatter

  ❌ ERROR
    Location: Line 48:40
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 60:30
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 76:32
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 91:36
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 115:41
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 150:46
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 197:45
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 223:34
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 242:29
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 259:31
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ⚠️  WARNING
    Location: Line 317:3
    Message: Unexpected dead URL `https://developer.adobe.com/commerce/frontend-core/guide/themes/js-mixins/`, expected live URL
    Rule: no-dead-urls

───────────────────────────────────────────────────────────────
📄 FILE: src/pages/shared-services/storefront-events/index.md
───────────────────────────────────────────────────────────────
  ⚠️  WARNING
    Location: Line 104:62
    Message: Unexpected dead URL `https://experienceleague.adobe.com/en/docs/commerce/live-search/live-search-admin/rules/rules-add#intelligent-ranking`, expected live URL
    Rule: no-dead-urls

  ⚠️  WARNING
    Location: Line 46:3
    Message: Unexpected dead URL `https://experienceleague.adobe.com/developer/commerce/storefront/setup/analytics/instrumentation/`, expected live URL
    Rule: no-dead-urls


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

  📁 Files processed:    5
  📄 Files with issues:  3
  ❌ Total errors:       10
  ⚠️  Total warnings:     6
  📋 Total issues:       16

Result: ❌ FAILED - Fatal errors found

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

This comment was automatically generated by the linter bot.

github-actions · 2026-06-24T14:14:25Z

🔍 Linter Report

Click to expand full report

═══════════════════════════════════════════════════════════════
                     LINTER REPORT
═══════════════════════════════════════════════════════════════

Generated: 2026-06-24T14:13:55.659Z
Mode: Full Linting (all rules + dead links check)
Target Directory: /home/runner/work/commerce-services/commerce-services
Skipped Rules:
  - check-frontmatter → src/pages/includes

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

Files to process: 5

───────────────────────────────────────────────────────────────
📄 FILE: src/pages/config.md
───────────────────────────────────────────────────────────────
  ⚠️  WARNING
    Location: Line 1:1
    Message: Missing frontmatter section - add --- at the beginning with title and description
    Rule: check-frontmatter

  ⚠️  WARNING
    Location: Line 5:7
    Message: Unexpected redirecting URL `https://developer.adobe.com/commerce/docs`, expected final URL `https://developer.adobe.com/commerce/docs/`
    Rule: no-dead-urls

───────────────────────────────────────────────────────────────
📄 FILE: src/pages/shared-services/implementation/tutorial.md
───────────────────────────────────────────────────────────────
  ⚠️  WARNING
    Location: Line 418:141
    Message: Cannot find heading for `#update-cdn`
    Rule: missing-heading

  ⚠️  WARNING
    Location: Line 2:1
    Message: Title is too long (71 characters). Consider keeping it under 60 characters.
    Rule: check-frontmatter

  ❌ ERROR
    Location: Line 48:40
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 70:30
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 91:32
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 112:36
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 152:40
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 234:50
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 376:45
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 420:47
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 452:42
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 483:31
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 525:55
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 551:52
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 570:39
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ⚠️  WARNING
    Location: Line 650:3
    Message: Unexpected dead URL `https://developer.adobe.com/commerce/frontend-core/guide/themes/js-mixins/`, expected live URL
    Rule: no-dead-urls

───────────────────────────────────────────────────────────────
📄 FILE: src/pages/shared-services/storefront-events/index.md
───────────────────────────────────────────────────────────────
  ⚠️  WARNING
    Location: Line 104:62
    Message: Unexpected dead URL `https://experienceleague.adobe.com/en/docs/commerce/live-search/live-search-admin/rules/rules-add#intelligent-ranking`, expected live URL
    Rule: no-dead-urls


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

  📁 Files processed:    5
  📄 Files with issues:  3
  ❌ Total errors:       13
  ⚠️  Total warnings:     6
  📋 Total issues:       19

Result: ❌ FAILED - Fatal errors found

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

This comment was automatically generated by the linter bot.

github-actions · 2026-06-24T14:46:44Z

🔍 Linter Report

Click to expand full report

═══════════════════════════════════════════════════════════════
                     LINTER REPORT
═══════════════════════════════════════════════════════════════

Generated: 2026-06-24T14:46:19.669Z
Mode: Full Linting (all rules + dead links check)
Target Directory: /home/runner/work/commerce-services/commerce-services
Skipped Rules:
  - check-frontmatter → src/pages/includes

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

Files to process: 5

───────────────────────────────────────────────────────────────
📄 FILE: src/pages/config.md
───────────────────────────────────────────────────────────────
  ⚠️  WARNING
    Location: Line 1:1
    Message: Missing frontmatter section - add --- at the beginning with title and description
    Rule: check-frontmatter

  ⚠️  WARNING
    Location: Line 5:7
    Message: Unexpected redirecting URL `https://developer.adobe.com/commerce/docs`, expected final URL `https://developer.adobe.com/commerce/docs/`
    Rule: no-dead-urls

───────────────────────────────────────────────────────────────
📄 FILE: src/pages/shared-services/implementation/tutorial.md
───────────────────────────────────────────────────────────────
  ⚠️  WARNING
    Location: Line 418:141
    Message: Cannot find heading for `#update-cdn-versions`
    Rule: missing-heading

  ⚠️  WARNING
    Location: Line 2:1
    Message: Title is too long (71 characters). Consider keeping it under 60 characters.
    Rule: check-frontmatter

  ❌ ERROR
    Location: Line 48:40
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 70:30
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 91:32
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 112:36
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 152:40
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 234:50
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 376:45
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 420:47
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 452:42
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 483:31
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 525:55
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 551:52
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

  ❌ ERROR
    Location: Line 570:39
    Message: Unescaped opening curly brace "{" found. Use "\{" to escape it.
    Rule: no-unescaped-curly-braces

───────────────────────────────────────────────────────────────
📄 FILE: src/pages/shared-services/storefront-events/index.md
───────────────────────────────────────────────────────────────
  ⚠️  WARNING
    Location: Line 104:62
    Message: Unexpected dead URL `https://experienceleague.adobe.com/en/docs/commerce/live-search/live-search-admin/rules/rules-add#intelligent-ranking`, expected live URL
    Rule: no-dead-urls


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

  📁 Files processed:    5
  📄 Files with issues:  3
  ❌ Total errors:       13
  ⚠️  Total warnings:     5
  📋 Total issues:       18

Result: ❌ FAILED - Fatal errors found

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

This comment was automatically generated by the linter bot.

github-actions · 2026-06-24T15:14:55Z

🔍 Linter Report

Click to expand full report

═══════════════════════════════════════════════════════════════
                     LINTER REPORT
═══════════════════════════════════════════════════════════════

Generated: 2026-06-24T15:14:31.645Z
Mode: Full Linting (all rules + dead links check)
Target Directory: /home/runner/work/commerce-services/commerce-services
Skipped Rules:
  - check-frontmatter → src/pages/includes

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

Files to process: 5

───────────────────────────────────────────────────────────────
📄 FILE: src/pages/config.md
───────────────────────────────────────────────────────────────
  ⚠️  WARNING
    Location: Line 1:1
    Message: Missing frontmatter section - add --- at the beginning with title and description
    Rule: check-frontmatter

  ⚠️  WARNING
    Location: Line 5:7
    Message: Unexpected redirecting URL `https://developer.adobe.com/commerce/docs`, expected final URL `https://developer.adobe.com/commerce/docs/`
    Rule: no-dead-urls

───────────────────────────────────────────────────────────────
📄 FILE: src/pages/shared-services/implementation/tutorial.md
───────────────────────────────────────────────────────────────
  ⚠️  WARNING
    Location: Line 2:1
    Message: Title is too long (71 characters). Consider keeping it under 60 characters.
    Rule: check-frontmatter

───────────────────────────────────────────────────────────────
📄 FILE: src/pages/shared-services/storefront-events/index.md
───────────────────────────────────────────────────────────────
  ⚠️  WARNING
    Location: Line 104:62
    Message: Unexpected dead URL `https://experienceleague.adobe.com/en/docs/commerce/live-search/live-search-admin/rules/rules-add#intelligent-ranking`, expected live URL
    Rule: no-dead-urls

  ⚠️  WARNING
    Location: Line 46:3
    Message: Unexpected dead URL `https://experienceleague.adobe.com/developer/commerce/storefront/setup/analytics/instrumentation/`, expected live URL
    Rule: no-dead-urls


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

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

Result: ⚠️  PASSED WITH WARNINGS - No fatal errors

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

This comment was automatically generated by the linter bot.

github-actions · 2026-06-24T19:27:33Z

🔍 Linter Report

Click to expand full report

═══════════════════════════════════════════════════════════════
                     LINTER REPORT
═══════════════════════════════════════════════════════════════

Generated: 2026-06-24T19:27:08.054Z
Mode: Full Linting (all rules + dead links check)
Target Directory: /home/runner/work/commerce-services/commerce-services
Skipped Rules:
  - check-frontmatter → src/pages/includes

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

Files to process: 5

───────────────────────────────────────────────────────────────
📄 FILE: src/pages/config.md
───────────────────────────────────────────────────────────────
  ⚠️  WARNING
    Location: Line 1:1
    Message: Missing frontmatter section - add --- at the beginning with title and description
    Rule: check-frontmatter

  ⚠️  WARNING
    Location: Line 5:7
    Message: Unexpected redirecting URL `https://developer.adobe.com/commerce/docs`, expected final URL `https://developer.adobe.com/commerce/docs/`
    Rule: no-dead-urls

───────────────────────────────────────────────────────────────
📄 FILE: src/pages/shared-services/implementation/tutorial.md
───────────────────────────────────────────────────────────────
  ⚠️  WARNING
    Location: Line 2:1
    Message: Title is too long (71 characters). Consider keeping it under 60 characters.
    Rule: check-frontmatter

───────────────────────────────────────────────────────────────
📄 FILE: src/pages/shared-services/storefront-events/index.md
───────────────────────────────────────────────────────────────
  ⚠️  WARNING
    Location: Line 104:62
    Message: Unexpected dead URL `https://experienceleague.adobe.com/en/docs/commerce/live-search/live-search-admin/rules/rules-add#intelligent-ranking`, expected live URL
    Rule: no-dead-urls


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

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

Result: ⚠️  PASSED WITH WARNINGS - No fatal errors

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

This comment was automatically generated by the linter bot.

github-actions · 2026-06-24T21:16:01Z

🔍 Linter Report

Click to expand full report

═══════════════════════════════════════════════════════════════
                     LINTER REPORT
═══════════════════════════════════════════════════════════════

Generated: 2026-06-24T21:15:32.615Z
Mode: Full Linting (all rules + dead links check)
Target Directory: /home/runner/work/commerce-services/commerce-services
Skipped Rules:
  - check-frontmatter → src/pages/includes

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

Files to process: 5

───────────────────────────────────────────────────────────────
📄 FILE: src/pages/config.md
───────────────────────────────────────────────────────────────
  ⚠️  WARNING
    Location: Line 1:1
    Message: Missing frontmatter section - add --- at the beginning with title and description
    Rule: check-frontmatter

  ⚠️  WARNING
    Location: Line 5:7
    Message: Unexpected redirecting URL `https://developer.adobe.com/commerce/docs`, expected final URL `https://developer.adobe.com/commerce/docs/`
    Rule: no-dead-urls

───────────────────────────────────────────────────────────────
📄 FILE: src/pages/shared-services/implementation/tutorial.md
───────────────────────────────────────────────────────────────
  ⚠️  WARNING
    Location: Line 2:1
    Message: Title is too long (71 characters). Consider keeping it under 60 characters.
    Rule: check-frontmatter

───────────────────────────────────────────────────────────────
📄 FILE: src/pages/shared-services/storefront-events/index.md
───────────────────────────────────────────────────────────────
  ⚠️  WARNING
    Location: Line 104:62
    Message: Unexpected dead URL `https://experienceleague.adobe.com/en/docs/commerce/live-search/live-search-admin/rules/rules-add#intelligent-ranking`, expected live URL
    Rule: no-dead-urls


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

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

Result: ⚠️  PASSED WITH WARNINGS - No fatal errors

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

This comment was automatically generated by the linter bot.

meker12

Some suggested updates. There are a lot of steps in the tutorial -- is it possible to chunk some of them together to make the TOC more scannable, and the required tasks easier to understand ---?

Co-authored-by: Margaret Eker <meker@adobe.com>

github-actions · 2026-06-25T12:29:16Z

🔍 Linter Report

Click to expand full report

═══════════════════════════════════════════════════════════════
                     LINTER REPORT
═══════════════════════════════════════════════════════════════

Generated: 2026-06-25T12:28:49.257Z
Mode: Full Linting (all rules + dead links check)
Target Directory: /home/runner/work/commerce-services/commerce-services
Skipped Rules:
  - check-frontmatter → src/pages/includes

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

Files to process: 5

───────────────────────────────────────────────────────────────
📄 FILE: src/pages/config.md
───────────────────────────────────────────────────────────────
  ⚠️  WARNING
    Location: Line 1:1
    Message: Missing frontmatter section - add --- at the beginning with title and description
    Rule: check-frontmatter

  ⚠️  WARNING
    Location: Line 5:7
    Message: Unexpected redirecting URL `https://developer.adobe.com/commerce/docs`, expected final URL `https://developer.adobe.com/commerce/docs/`
    Rule: no-dead-urls

───────────────────────────────────────────────────────────────
📄 FILE: src/pages/shared-services/implementation/tutorial.md
───────────────────────────────────────────────────────────────
  ⚠️  WARNING
    Location: Line 2:1
    Message: Title is too long (71 characters). Consider keeping it under 60 characters.
    Rule: check-frontmatter

───────────────────────────────────────────────────────────────
📄 FILE: src/pages/shared-services/storefront-events/index.md
───────────────────────────────────────────────────────────────
  ⚠️  WARNING
    Location: Line 104:62
    Message: Unexpected dead URL `https://experienceleague.adobe.com/en/docs/commerce/live-search/live-search-admin/rules/rules-add#intelligent-ranking`, expected live URL
    Rule: no-dead-urls


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

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

Result: ⚠️  PASSED WITH WARNINGS - No fatal errors

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

This comment was automatically generated by the linter bot.

Adding new tutorial to commerce-services

a8bc809

Copilot AI review requested due to automatic review settings June 23, 2026 17:03

erikmarr self-assigned this Jun 23, 2026

erikmarr added the new-topic A major update published as an entirely new document label Jun 23, 2026

dshevtsov added this to Commerce - Pull Requests Jun 23, 2026

github-project-automation Bot moved this to 📋 Needs Review in Commerce - Pull Requests Jun 23, 2026

Copilot started reviewing on behalf of erikmarr June 23, 2026 17:04 View session

Copilot AI reviewed Jun 23, 2026

View reviewed changes

Comment thread src/pages/shared-services/storefront-events/index.md Outdated

Comment thread src/pages/shared-services/implementation/tutorial.md Outdated

Comment thread src/pages/shared-services/implementation/tutorial.md

Minor clean up

80519ce

Erik Marr added 2 commits June 23, 2026 14:19

restoring code snippets

e34cfff

Fixing validation errors

bf73cfa

adding back anchor links

be71b81

fixing linting errors again

b7a8af7

entering review comments

a004acf

chore: auto-generate contributors

2a147bc

meker12 approved these changes Jun 24, 2026

View reviewed changes

github-project-automation Bot moved this from 📋 Needs Review to 👍 Approved in Commerce - Pull Requests Jun 24, 2026

Apply suggestions from code review

d9b745f

Co-authored-by: Margaret Eker <meker@adobe.com>

erikmarr merged commit 3a4c717 into main Jun 25, 2026
6 checks passed

erikmarr deleted the em_COMOPT-2021 branch June 25, 2026 12:29

github-project-automation Bot moved this from 👍 Approved to 🏁 Done in Commerce - Pull Requests Jun 25, 2026

Uh oh!

Conversation

erikmarr commented Jun 23, 2026

Purpose of this pull request

Affected pages

Uh oh!

github-actions Bot commented Jun 23, 2026

🔍 Linter Report

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Jun 23, 2026

🔍 Linter Report

Uh oh!

github-actions Bot commented Jun 24, 2026

🔍 Linter Report

Uh oh!

github-actions Bot commented Jun 24, 2026

🔍 Linter Report

Uh oh!

github-actions Bot commented Jun 24, 2026

🔍 Linter Report

Uh oh!

github-actions Bot commented Jun 24, 2026

🔍 Linter Report

Uh oh!

github-actions Bot commented Jun 24, 2026

🔍 Linter Report

Uh oh!

meker12 left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Jun 25, 2026

🔍 Linter Report

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

meker12 left a comment •

edited

Loading