five23 / shipsync-community

FedEx Web Services Integration for Magento
9 stars 10 forks source link

Length, Width, and Height attributes not installing with ShipSync V5 #12

Closed hgomes13 closed 10 years ago

hgomes13 commented 10 years ago

CE 1.7.0.2 with custom theme. installing V5 doesn't seem to add the attributes "length, height, and width" to our Magento installation. Installing v5 on a fresh instal of CE 1.7.0.2 however works, and on a fresh CE 1.8.1.0

this inherently breaks the rate request to FedEx to get rates based on dimensions. I've tried installing the attributes manually, but that doesn't fix it. (shot in the dark)

I've taken a look into the source and I think there are 2 problems with it: typo on "package.php" line 263 missing a "v" in value; but else I have no clue how the EAV / attribute dependencies work or whether there is a error log to check.

five23 commented 10 years ago

Verified bug.

I'll include a fix in the next commit (as well as confirm backwards compatibility with 1.7)

hgomes13 commented 10 years ago

Greatly appreciated guys! When will the next commit be?

GBVM commented 10 years ago

Looking forward to seeing a fix for this as well. Can you please let me know an estimated time for this. Im willing to pay to have this taken care of right away.

hgomes13 commented 10 years ago

Any luck with the bug fix?

five23 commented 10 years ago

Hey guys, sorry this has been delayed ... I'm sooooo close now to a stable version, and then I'll be focusing on backwards compatibility.

I can do a custom mod to get you guys going in the meantime if needed. Email me : david@ecomatics.net.

Otherwise, it'll probably be another week or so before I start testing with Magento 1.5-1.7

five23 commented 10 years ago

ShipSync is working properly on Magento 1.7.x now, and presumably should now work with versions as far back as 1.5.x.