Skip to content

Add staging mode configuration to deployment settings#990

Merged
Soner (shyim) merged 1 commit intomainfrom
claude/add-json-schema-fields-44xSU
May 10, 2026
Merged

Add staging mode configuration to deployment settings#990
Soner (shyim) merged 1 commit intomainfrom
claude/add-json-schema-fields-44xSU

Conversation

@shyim
Copy link
Copy Markdown
Member

Summary

This PR adds support for staging mode configuration in the deployment settings, allowing users to control whether staging setup commands are executed during installation and upgrade processes.

Key Changes

  • Added ConfigDeploymentStaging struct to internal/shop/config.go with an Enabled boolean field to control staging setup command execution
  • Extended ConfigDeployment struct to include an optional Staging field referencing the new staging configuration
  • Updated the JSON schema (shopware-project-schema.json) to include the new ConfigDeploymentStaging definition with proper type and description
  • Added staging property to the deployment configuration schema
  • Minor documentation improvement: removed "locally" from asset caching description for clarity

Implementation Details

  • The Staging field is optional (uses omitempty tag) to maintain backward compatibility
  • The staging configuration is defined as a separate struct for better organization and potential future extensibility
  • Both the Go code and JSON schema have been updated consistently with matching descriptions

https://claude.ai/code/session_01CHUaF3xNhgHWHbj13KetiD

Adds ConfigDeploymentStaging struct and deployment.staging.enabled field
to mirror the staging support added in shopware/deployment-helper#83.

https://claude.ai/code/session_01CHUaF3xNhgHWHbj13KetiD
@shyim Soner (shyim) merged commit 7d49478 into main May 10, 2026
3 checks passed
@shyim Soner (shyim) deleted the claude/add-json-schema-fields-44xSU branch May 10, 2026 14:26
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