Skip to content

Fix OpenSearch admin docs: prerequisites, capitalization, and env var index#2202

Merged
Vin Le (vienthuong) merged 2 commits intofeat/update-global-admin-es-integrationfrom
copilot/sub-pr-2195
Mar 20, 2026
Merged

Fix OpenSearch admin docs: prerequisites, capitalization, and env var index#2202
Vin Le (vienthuong) merged 2 commits intofeat/update-global-admin-es-integrationfrom
copilot/sub-pr-2195

Conversation

Copy link
Contributor

Copilot AI commented Mar 20, 2026

Five documentation issues in the OpenSearch admin feature docs, all stemming from missing prerequisite context and inconsistent terminology.

Changes

  • elasticsearch-setup.md

    • Fixed placeholder casing: YOUR OPEN SEARCH URLYOUR OPENSEARCH URL
    • Added prerequisite info block to the "Apply OpenSearch globally in Admin API" section — clarifies that ADMIN_OPENSEARCH_URL and SHOPWARE_ADMIN_ES_ENABLED=1 must be set before ENABLE_OPENSEARCH_FOR_ADMIN_API=1 has any effect
  • performance-tweaks.md

    • Standardized section heading and intro: Elasticsearch/OpensearchElasticsearch/OpenSearch
    • Updated ENABLE_OPENSEARCH_FOR_ADMIN_API paragraph to explicitly state the Admin OpenSearch prerequisite before recommending the flag
  • environment-variables.md

    • Added ENABLE_OPENSEARCH_FOR_ADMIN_API to the env var reference table (alphabetically ordered), with prerequisites, experimental status, and since 6.7.9.0 noted

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: vienthuong <22548423+vienthuong@users.noreply.github.com>
Copilot AI changed the title [WIP] [WIP] Address feedback on applying Opensearch globally in admin docs Fix OpenSearch admin docs: prerequisites, capitalization, and env var index Mar 20, 2026
@vienthuong Vin Le (vienthuong) marked this pull request as ready for review March 20, 2026 12:26
@vienthuong Vin Le (vienthuong) merged commit 3824567 into feat/update-global-admin-es-integration Mar 20, 2026
1 check failed
@vienthuong Vin Le (vienthuong) deleted the copilot/sub-pr-2195 branch March 20, 2026 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants