Skip to content
This repository has been archived by the owner on Jun 13, 2018. It is now read-only.

Removing UPS integration #544

Open
maartenvg opened this issue Oct 10, 2017 · 10 comments
Open

Removing UPS integration #544

maartenvg opened this issue Oct 10, 2017 · 10 comments

Comments

@maartenvg
Copy link
Contributor

maartenvg commented Oct 10, 2017

WIP

@maartenvg maartenvg reopened this Oct 10, 2017
@jimbert
Copy link

jimbert commented Oct 29, 2017

Why do you plan to remove UPS integration?

@holman
Copy link

holman commented Nov 15, 2017

Why this change? I mean, obviously UPS is a terrible organization, but I don't see any reason why including a wrapper around their API — that you already have to sign up for — would pose problems.

Certainly you're still using these codepaths at Shopify; going back and wiping the entire project of any trace of UPS is pretty ridiculous.

@dhonig
Copy link

dhonig commented Nov 18, 2017

what is the motivation for this change? Was there some kind of legal issue in having it remain?

@jtokoph
Copy link

jtokoph commented Nov 23, 2017

Does anyone have a fork/clone with the original history?

@dhonig
Copy link

dhonig commented Nov 23, 2017 via email

@tblanchard
Copy link

Super annoying, why was this done?

@BenAkroyd
Copy link

I would also like to know the reason UPS requested the removal, and why the git history was rewritten to remove all trace of the code

@mikeswimm
Copy link

This is absurd. I sent an email to my rep this morning. I will share anything I learn.

@sabotatore
Copy link

@maartenvg What is the status of 2.2.0 gem version? Are you going to release it?

@jiblits
Copy link
Contributor

jiblits commented May 29, 2018

This might be why.

There's still gems with the old code, so a fork or extension of active_shipping that isn't associated with Shopify would probably need to be created where this code can live.

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

No branches or pull requests

10 participants