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.

CommentFileSizeAuthor
#2 prefix_module_name-3145745-2.patch319 bytesmorbus iff
Screen Shot 2020-06-04 at 8.17.18 AM.png70.86 KBmorbus iff

Comments

Morbus Iff created an issue. See original summary.

morbus iff’s picture

morbus iff
they/them
commented
Status: Active » Needs review
StatusFileSize
new prefix_module_name-3145745-2.patch319 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

morbus iff
they/them
commented
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

mglaman
Primary language English
Location WI, USA
commented
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.