Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Carrier is not detected for some UPS tracking numbers #26

Closed
shkrt opened this issue Apr 4, 2019 · 8 comments
Closed

Carrier is not detected for some UPS tracking numbers #26

shkrt opened this issue Apr 4, 2019 · 8 comments

Comments

@shkrt
Copy link

shkrt commented Apr 4, 2019

There are UPS tracking numbers that look like this K1506235620.

Expected behaviour:

TrackingNumber.new('K1506235620').carrier
=> :ups

Actual behaviour:

TrackingNumber.new('K1506235620').carrier
=> :unknown

version used: 1.0.5

@jkeen jkeen transferred this issue from jkeen/tracking_number Apr 4, 2019
@jkeen
Copy link
Owner

jkeen commented Apr 4, 2019

I don't have any record of UPS numbers besides the ones starting with 1Z. You're sure this is a tracking number and not some other number on the label?

@shkrt
Copy link
Author

shkrt commented Apr 4, 2019

I got that number from one of my customers, and package is actually trackable using it, as seen on the attached image:

image

(I intentionally painted out some info for the sake of privacy). But I also don't see that kind of numbers anywhere in the documentation.

Also, this code gives us a clue that there is also UPS tracking numbers starting with H

@shkrt
Copy link
Author

shkrt commented Apr 12, 2019

@tleish
Copy link

tleish commented Aug 14, 2019

It appears the code comes from a UPS handwritten labels, also called a UPS Worldwide Services Waybill. I've seen several posts where it can start with A, H, J, K and T.

See:

Another shipping website specializing in detecting tracking numbers identified 'K1506235620' (the original posters) as 1 of 4 carriers: UPS, Delnext, GLS Italy or Total Express.

When testing, it shows the same list of 4 possible carriers for anything [A-Z]1506235620.

jkeen added a commit that referenced this issue Nov 3, 2022
jkeen added a commit that referenced this issue Nov 3, 2022
@jkeen
Copy link
Owner

jkeen commented Nov 3, 2022

Added support for these numbers in #59

@jkeen jkeen closed this as completed Nov 3, 2022
@AdamAld
Copy link
Contributor

AdamAld commented Dec 22, 2022

Added additional support for UPS WorldWide Express Saver ("V"+10 digits) in #75

@digital-marketing-engineer

We need to have UPS documentation reference of their format.

@jkeen
Copy link
Owner

jkeen commented May 17, 2023

We need to have UPS documentation reference of their format.

I'm not sure what you're asking here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants