Skip to content

Remove the shop configuration billing option of the carrier form - #1111

Merged
Progi1984 merged 1 commit into
PrestaShop:mainfrom
jolelievre:remove-carrier-shop-configuration-billing
Aug 25, 2026
Merged

Remove the shop configuration billing option of the carrier form#1111
Progi1984 merged 1 commit into
PrestaShop:mainfrom
jolelievre:remove-carrier-shop-configuration-billing

Conversation

@jolelievre

Copy link
Copy Markdown
Contributor

Why

Following the QA review of PrestaShop/PrestaShop#42022, the Based on the shop configuration shipping method is removed from the carrier form: the underlying PS_SHIPPING_METHOD configuration has had no back office interface since 2013 and is now deprecated for removal in the next major.

What

The value-based selectors of the price and weight radios introduced by #1102 are kept: they remain the robust way to target the choices whatever their order.

PrestaShop/PrestaShop#42022 no longer offers 'Based on the shop
configuration' as a shipping method: the underlying PS_SHIPPING_METHOD
configuration is a read-only migration marker deprecated for removal.
The dedicated radio selector, the billing branch and the
getBillingSelection getter introduced for that option are removed. The
value-based selectors of the price and weight radios are kept, they
remain the robust way to target the choices whatever their order.
@jolelievre
jolelievre requested a review from a team as a code owner August 25, 2026 11:47
@github-project-automation github-project-automation Bot moved this to Ready for review in PR Dashboard Aug 25, 2026
@Progi1984
Progi1984 merged commit 5855bec into PrestaShop:main Aug 25, 2026
2 checks passed
@Progi1984 Progi1984 added this to the 0.0.12 milestone Aug 25, 2026
@github-project-automation github-project-automation Bot moved this from Ready for review to Merged in PR Dashboard Aug 25, 2026
@ps-jarvis ps-jarvis moved this from Merged to Ready for review in PR Dashboard Aug 25, 2026
@jolelievre
jolelievre deleted the remove-carrier-shop-configuration-billing branch August 25, 2026 11:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Ready for review

Development

Successfully merging this pull request may close these issues.

3 participants