Skip to content

feat: shipping express toggle#586

Merged
Michel (cyl3x) merged 6 commits intotrunkfrom
feat/shipping-callback-toggle
Mar 30, 2026
Merged

feat: shipping express toggle#586
Michel (cyl3x) merged 6 commits intotrunkfrom
feat/shipping-callback-toggle

Conversation

@cyl3x
Copy link
Copy Markdown
Contributor

1. Why is this change necessary?

There are some troubles with the shipping callback we implemented.
You only can disable it implicitly and it needs the store-api to be exposed.

2. What does this change do, exactly?

  • Add a dedicated setting for the shipping callback
  • Add a proxy storefront route so the store-api doesn't need to be exposed
  • Disable settings based on "Local environment" active incl. tooltips
  • Fix reloading the webhook status on settings save

3. Describe each step to reproduce the issue or behaviour.

4. Please link to the relevant issues (if any).

5. Checklist

  • I have written tests and verified that they fail without my change
  • I have created an entry in the CHANGELOG.md files with all necessary user information about my changes
  • This change has comments for package types, values, functions, and non-obvious lines of code
  • I have read the contribution requirements and fulfill them.

@cyl3x Michel (cyl3x) self-assigned this Mar 21, 2026
@cyl3x Michel (cyl3x) marked this pull request as ready for review March 24, 2026 07:44
@cyl3x Michel (cyl3x) requested review from a team and Fabian Boensch (En0Ma1259) and removed request for Fabian Boensch (En0Ma1259) March 24, 2026 07:44
Comment thread src/Resources/app/administration/src/app/snippet/de.json Outdated
Comment thread src/Resources/app/administration/src/app/snippet/en.json Outdated
@cyl3x Michel (cyl3x) added this pull request to the merge queue Mar 30, 2026
Merged via the queue into trunk with commit cc28a39 Mar 30, 2026
26 checks passed
@cyl3x Michel (cyl3x) deleted the feat/shipping-callback-toggle branch March 30, 2026 13:44
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