verbb / postie

A Craft Commerce shipping calculator plugin.
Other
12 stars 18 forks source link

[4.0.3] UPS Rates Different / Service detail pages broken #132

Open RitterKnightCreative opened 6 days ago

RitterKnightCreative commented 6 days ago

Describe the bug

Working through the process of upgrading Postie from 3 -> 4.

The upgrade itself seemed to work, though I had to put in the carrier (UPS), box details, and credentials all over again. Not sure if that was supposed to taken care of in the migration or not? No big deal either way.

However, I also noticed that when I go into each service (eg Ground), I'm getting a 404 not found error. Maybe a known issue?

More imporantly, I'm getting vastly different rates between versions, as if our clients' negotiated rates aren't showing up. Assuming the box packer itself hasn't been tweaked, I'm not sure what else I'm missing?

Screenshot 2024-06-28 at 3 11 25 PM

Current Prodution (Postie 3) Screenshot 2024-06-28 at 3 05 23 PM

Current testing (Postie 4) Screenshot 2024-06-28 at 3 08 15 PM

Steps to reproduce

  1. Upgrade Postie from 3 to 4.
  2. Setup a new carrier (UPS), box sizes, etc.

Craft CMS version

4.10.3

Plugin version

4.0.3

Multi-site?

Yes

Additional context

No response

engram-design commented 6 days ago

Ah, sorry about that. I didn't factor in the already-updated OAuth-based provider, just the older non-OAuth one. Updated for the next release.

As for the 404, that's because we switched handles for services back to their numeric ones. Other providers use strings, and that's causing issues for the route matching.

Fixed for the next release. To get this early, run composer require verbb/postie:"dev-craft-4 as 4.0.3".

That's quite interesting! Can you send through the test info you're using (from, to, dimensions/weights). Seems like you're using the front-end to test, so either the from/to address and the contents of your cart, or send through a screenshot of the same info in the "Rates Testing" tab in the control panel for the provider.

I don't think I'll need your connection credentials and account number for negotiated rates yet. You're also welcome to send via support@verbb.io if you're rather send anything privately.

RitterKnightCreative commented 3 days ago

Excellent, thank you!

I did a little more digging and I'm wondering if UPS is changing the address classification? I'm getting this returned from UPS: [Code] => 110920 [Description] => Ship To Address Classification is changed from Commercial to Residential

The weird thing is I'm not getting that with the older version. Wonder if there was any address correction going on? (I'd have to test with a commercial address to see.)

This might also explain why my client had to put a 65% markup on the current site if the rate getting spit out is too low...

RESPONSE.txt REQUEST.txt