Skip to content

[docs] Clarify that -e/--env now applies to all Admin-API commands - #2458

Open
shopware-docs-updater[bot] wants to merge 2 commits into
mainfrom
docs/pr-1257-env-flag-admin-api-47aa766e5550a3dc
Open

[docs] Clarify that -e/--env now applies to all Admin-API commands#2458
shopware-docs-updater[bot] wants to merge 2 commits into
mainfrom
docs/pr-1257-env-flag-admin-api-47aa766e5550a3dc

Conversation

@shopware-docs-updater

Copy link
Copy Markdown

Related to shopware/shopware-cli#1257 (fix by @moshimorschi).

User-facing change

That PR fixed all project Admin-API commands (extension list, extension uninstall, extension install, extension update, extension outdated, extension upload, extension delete, extension activate, extension deactivate, admin-api, clear-cache, upgrade check) to honor the global -e/--env flag and resolve the target environment from .shopware-project.yml. Previously, only executor-based commands (admin-build, ci, dev, ...) honored the flag; Admin-API commands silently used the base config, and an unknown environment name was silently ignored instead of causing a failure.

Documentation gap

The docs did not mention that -e/--env applies to Admin-API commands, and did not document the "unknown environment fails" behavior at all.

Files changed

  • docs/guides/development/dev-environment.md — noted that -e/--env selects the environment for all commands that talk to the shop (not just executor commands), and that an unknown name causes a failure.
  • docs/products/tools/cli/project-commands/remote-extension-management.md — added a short "Targeting an environment" section with an example (shopware-cli project extension list -e staging) and a link back to the environment executors reference.

Please review for accuracy against the merged behavior.

Generated by 📝 PR documentation check for #1257 · auto · 32.2 AIC · ⌖ 2.64 AIC · ⊞ 7.6K ·

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@shopware-dev-docs-connector

shopware-dev-docs-connector Bot commented Aug 18, 2026

Copy link
Copy Markdown

Developer Docs healthcheck

Status: Completed with failure.
Repository: shopware/docs
Commit: 5d5c591
Preview: No Vercel preview URL was found in the workflow logs.
Workflow run: #4877

@shyim
Soner (shyim) marked this pull request as ready for review August 18, 2026 10:47
@shyim
Soner (shyim) requested review from Micha Hobert (Isengo1989) and a lite review from Copilot August 18, 2026 10:47
@github-actions

Copy link
Copy Markdown
Contributor

📢 Developer Announcement Recommendation

Recommendation: ❌ No Announcement Recommended

No developer announcement signals were detected.

This PR does not appear to introduce developer-facing changes that require a community announcement.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the Shopware CLI documentation to reflect that the global -e/--env flag applies to all project commands that communicate with a shop (including Admin API–based commands), and that specifying an unknown environment now fails instead of silently falling back to the default configuration.

Changes:

  • Documented that -e/--env is honored by Admin API command families (not only executor-based commands).
  • Added explicit guidance + an example for selecting a named environment from .shopware-project.yml.
  • Documented the new failure behavior for unknown environment names.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
products/tools/cli/project-commands/remote-extension-management.md Adds a “Targeting an environment” section describing -e/--env, how it selects environments.<name> config, and the unknown-env failure behavior.
guides/development/dev-environment.md Clarifies in the environment executors section that -e/--env affects Admin API command families too, and that unknown environment names now error.

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

@Isengo1989 Micha Hobert (Isengo1989) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM 👍

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.

3 participants