Skip Menu |

This queue is for tickets about the Net-UPS CPAN distribution.

Report information
The Basics
Id: 92473
Status: resolved
Priority: 0/
Queue: Net-UPS
People
Owner: Nobody in particular
Requestors: perl [...] xev.net
Cc:
AdminCc:
Bug Information
Severity: Wishlist
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Sat Jan 25 18:05:25 2014 perl [...] xev.net - Ticket created
Subject: add support for street level address validation
I added support for street level address validation. The code changes are at: https://github.com/xevo/Net-UPS If you like the changes then please integrate them with your module. Thanks for the great module!
  Sat Jan 25 22:05:24 2014 perl [...] xev.net - Correspondence added
Subject: diff
Here are the changes: https://github.com/xevo/Net-UPS/compare/9c28b70b6b586ced28ceb3872015d07cadcecc25...7ab89dd50c7de74d31defb2fbdaa1856baf64961 Sorry, I committed the 0.04 branch after everything else so the changes are actually showing up in red instead of green. I you like, I can submit a real patch.
  Sat Jan 25 22:48:03 2014 perl [...] xev.net - Correspondence added
On Sat Jan 25 22:05:24 2014, KEN wrote: Show quoted text
> Here are the changes: > > https://github.com/xevo/Net- > UPS/compare/9c28b70b6b586ced28ceb3872015d07cadcecc25...7ab89dd50c7de74d31defb2fbdaa1856baf64961 > > Sorry, I committed the 0.04 branch after everything else so the > changes are actually showing up in red instead of green. I you like, I > can submit a real patch.
Ignore my last post. I fixed the repo, here is the correct diff: https://github.com/xevo/Net-UPS/compare/772e58a3a01affb522e55fbdae18f866d99982cc...48b81fdad5b906863944b2942d3e0d67022fd981 And I also attached a patch.
Subject: net-ups-street-level-patch.diff

Message body is not shown because it is too large.

  Sun Jan 26 22:20:35 2014 perl [...] xev.net - Correspondence added
I made a few more changes to support up to three address fields as well as the building name. I also added some tests for the street level api. The latest version will always be here: https://github.com/xevo/Net-UPS/compare/0.04...master
  Thu Feb 27 06:26:59 2014 DAKKAR [...] cpan.org - Correspondence added
I'm manually merging the code to current master, will release in a bit.
  Thu Feb 27 06:27:01 2014 The RT System itself - Status changed from 'new' to 'open'
  Thu Feb 27 06:45:41 2014 DAKKAR [...] cpan.org - Correspondence added
released 0.11 with the new code; there is no separate StreetAddress class, I just extended the Address class to work for all requests.
  Thu Feb 27 06:45:42 2014 DAKKAR [...] cpan.org - Status changed from 'open' to 'resolved'