Skip to content

feat(ecom-gateway): DOMA-13399 add middleware routes#7824

Merged
AleX83Xpert merged 10 commits into
mainfrom
feat/ecom-gateway/doma-13399/add-middleware-routes
Jul 9, 2026
Merged

feat(ecom-gateway): DOMA-13399 add middleware routes#7824
AleX83Xpert merged 10 commits into
mainfrom
feat/ecom-gateway/doma-13399/add-middleware-routes

Conversation

@AleX83Xpert

@AleX83Xpert AleX83Xpert commented Jul 7, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Chores
    • Updated the integrated e-commerce gateway component to a newer revision to keep it aligned with the latest supported version, including upstream improvements and fixes.

@AleX83Xpert AleX83Xpert force-pushed the feat/ecom-gateway/doma-13399/add-middleware-routes branch from dd20db3 to bb152ff Compare July 7, 2026 07:04
@coderabbitai

coderabbitai Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review
📝 Walkthrough

Walkthrough

The pull request updates the pinned commit reference for the apps/ecom-gateway git submodule from one commit hash to another. No other code changes are present.

Changes

Submodule Pointer Update

Layer / File(s) Summary
Update ecom-gateway submodule reference
apps/ecom-gateway
The subproject commit reference for apps/ecom-gateway is bumped from db611aa5efb58291604bf19b1feb1173e609d880 to 22c609e684cd03991ae70acf2007a7d5b3763d3e.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested labels: ✋🙂 Review please

Suggested reviewers: abshnko, dkoviazin, sitozzz

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title says middleware routes were added, but the diff only updates the ecom-gateway subproject commit reference. Rename the title to match the actual change, or include the middleware route changes if they were omitted from the diff.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/ecom-gateway/doma-13399/add-middleware-routes

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@AleX83Xpert AleX83Xpert force-pushed the feat/ecom-gateway/doma-13399/add-middleware-routes branch from aca4fc6 to 3a399f8 Compare July 9, 2026 04:24
move ECOM_SERVER_URL configuration before OIDC setup in prepare script
…quest handler and fix transformIgnorePatterns spacing
@sonarqubecloud

sonarqubecloud Bot commented Jul 9, 2026

Copy link
Copy Markdown

@AleX83Xpert AleX83Xpert merged commit cc60501 into main Jul 9, 2026
126 of 129 checks passed
@AleX83Xpert AleX83Xpert deleted the feat/ecom-gateway/doma-13399/add-middleware-routes branch July 9, 2026 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant