Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Shipping methods link is in the root of the commerce menu but it should probably go under Configuration > Shipping.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | shipping-methods-link-3156717-1.patch | 513 bytes | rinasek |
| |||
Comments
Comment #2
rinasek CreditAttribution: rinasek at Centarro commentedPatch attached
Comment #4
jsacksick CreditAttribution: jsacksick at Centarro commentedCommitted!
Comment #5
bojanz CreditAttribution: bojanz at Centarro commentedNote that this reverts #3076739: Shipping methods shouldn't be under /admin/commerce/config when they're not config fixed for 2.0-rc1. I suggest adding a comment to that issue explaining the revert.
You'll also want to make sure shipping method translation still works, I don't think we have test coverage for it.
Comment #6
bojanz CreditAttribution: bojanz at Centarro commentedBumping status for the translation question in #5
Comment #7
jsacksick CreditAttribution: jsacksick at Centarro commentedOh you're right... It should still work since we're not changing the paths... (which by the way is causing the breadcrumb to be broken).
I may have committed this too early without thinking of the side effects, it's probably safer to rollback this before the next release which is going to happen really soon and come back to this later...
Comment #9
jsacksick CreditAttribution: jsacksick at Centarro commentedPostponing this until we add tests that demonstrate that the translation UI is still working (it probably can't work under config because the url becomes then too large for Drupal which only supports 9 levels/parts, and not 10).