Sync develop with main branch #288
Conversation
feat: replace staging template ID and server URL with production sandbox
Replace tenant ID with instance ID
Data Ingestion REST API playground (production)
There was a problem hiding this comment.
Pull Request Overview
This PR synchronizes the develop branch with the main branch, updating API reference documentation and standardizing terminology from "tenant ID" to "instance ID" throughout the Adobe Commerce Optimizer documentation.
- Standardizes terminology from "tenant ID" to "instance ID" across all documentation files
- Updates API reference implementation to use a new credential component and local schema file
- Refactors OpenAPI schema parameter ordering and server configuration
Reviewed Changes
Copilot reviewed 11 out of 13 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| static/rest/data-ingestion-schema-v1.yaml | Updates server configuration to use parameterized tenantId and reorders API parameters |
| src/pages/reference/rest/index.md | Replaces external schema reference with local file and adds credential component |
| src/pages/optimizer/*/using-the-api.md | Updates terminology from "tenant ID" to "instance ID" in API documentation |
| src/pages/optimizer/*/troubleshooting.md | Updates error message terminology to use "instance ID" |
| src/pages/optimizer/*/api-reference.md | Removes standalone API reference pages |
| src/pages/credential/GetCredentialOAuthS2s.js | Adds new React component for OAuth credential management |
| src/openapi/.redocly.yaml | Removes Redocly configuration file |
| src/_includes/authentication/get-tenant-id.md | Updates authentication documentation to use "instance ID" terminology |
|
Sync PRs are not supposed to have editorial reviews. Direct merges between protected branches are not allowed. |
Fix typo
Streamline description.
Fix syntax.
Small corrections.
dshevtsov
left a comment
dshevtsov
left a comment
There was a problem hiding this comment.
Sync PRs are not supposed to have editorial reviews. Direct merges between protected branches are not allowed.
We use different best practice to sync main with develop. See example at AdobeDocs/commerce-php#383.
The branch protection needs to be revisited in this repo.
github-project-automation
bot
moved this from 📋 Needs Review
to 🛠 Changes Requested
in Commerce - Pull Requests
github-project-automation
bot
moved this from 🛠 Changes Requested
to 🏁 Done
in Commerce - Pull Requests
Purpose of this pull request
Sync develop with main branch