This modules needs to be updated to work with the 2.x branch of the Commerce Shipping module.

CommentFileSizeAuthor
#1 commerce-shipping-2x-branch-1348088-1.patch15.28 KBjantoine

Comments

jantoine’s picture

jantoine commented
Status: Active » Needs review
StatusFileSize
new commerce-shipping-2x-branch-1348088-1.patch15.28 KB

I have pretty much completely re-written the module using the 1.x as a base along with code from the Commerce Shipping and Commerce UPS modules. I have attached a patch. I would gladly help create/maintain the 2.x branch if I were added as a maintainer!

jantoine’s picture

jantoine commented

To match all other commerce modules, perhaps we should also implement this 2.x branch under the commerce_usps project name? Looking for consensus before I go creating a new project.

rszrama’s picture

rszrama commented

Yeah, I don't see any reason to have _shipping in all these plugins; we don't do the same for payment method modules (i.e. it's commerce_paypal, not commerce_payment_paypal). It's self-evident that a USPS module will be for shipping, so I don't see a need to extend the length of the module's function names.

spoonbow’s picture

spoonbow commented
Priority: Major » Normal

Is there anything happening with this module? Still being developed?

jantoine’s picture

jantoine commented

I haven't had a chance to create the project yet, but given Ryan's blessing, I will probably move forward with this shortly after the holidays if someone else doesn't beat me too it! Until then, the patch is working for me.

rszrama’s picture

rszrama commented

Excellent, can't wait to test it out. : )

logickal’s picture

logickal commented

Thanks for the patch! We are actually getting back in to development of this module, and can roll the 2.x branch in in the next couple of days if another project hasn't been created yet.

logickal’s picture

logickal commented

I've applied AntoineSolution's patch in #1 to the 7.x-2.x branch and committed. Still to be addressed is renaming to commerce_usps, but that should probably be another issue.

jrose’s picture

jrose commented

I'm am fairly new to drupal and believe this patch is what I'm looking for. Can anyone explain in newbie terms how to install the patch?

Thanks.

andyg5000’s picture

andyg5000
Primary language English
Location North Carolina, USA
commented
Status: Needs review » Closed (fixed)

The 7.x-2.x module has been renamed and ported to Commerce USPS http://drupal.org/project/commerce_usps. Closing issue.

rszrama’s picture

rszrama commented

Woohoo! : )