Currently, the commerce_shipping.info.yml uses "Shipping" as a module name. This is in stark contrast to every other Commerce module (which uses the "Commerce" prefix) as well as common shipping modules like Commerce UPS, Commerce USPS, and Commerce FedEx. See screenshot for the visual disparity.

Comments

Morbus Iff created an issue. See original summary.

morbus iff’s picture

Status: Active » Needs review
StatusFileSize
new319 bytes

Status: Needs review » Needs work

The last submitted patch, 2: prefix_module_name-3145745-2.patch, failed testing. View results

morbus iff’s picture

Status: Needs work » Reviewed & tested by the community

The two failed tests are unrelated to this issue.

  • mglaman committed 99204ca on 8.x-2.x authored by Morbus Iff
    Issue #3145745 by Morbus Iff: .info.yml should use "Commerce Shipping"...
mglaman’s picture

Assigned: morbus iff » Unassigned
Status: Reviewed & tested by the community » Fixed

Works for me and normalizes things with other ecosystem modules. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.