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
🚧 ⚰️ Authorize.net is dead #3224
Conversation
Well old pods are deleted and new pods are started with a new server. |
class RemoveAuthorizeNetCCtokenFromAccount < ActiveRecord::Migration[5.2] | ||
def change | ||
safety_assured { remove_column :accounts, :credit_card_authorize_net_payment_profile_token } | ||
end | ||
end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
According to this: https://github.com/ankane/strong_migrations#removing-a-column
The app should be restarted to be sure this migration doesn't cause problems, def. we'll need ops assistance to deploy this. But to be sure it doesn't cause any problem, another PR with only the self.ignored_columns = %i[credit_card_authorize_net_payment_profile_token]
added to the Account
model should be merged and deployed first.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
...per_portal/app/views/developer_portal/admin/account/authorize_net/credit_card_error.html.erb
Show resolved
Hide resolved
@@ -1,12 +0,0 @@ | |||
module Liquid |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess we need to leave it here, the same way we did with the ogone form liquid.
This needs quite some work to resolve all conflicts 🥲 Maybe move it to Draft for now as it's not a priority? |
This one makes a year in a couple of days! |
afefcc1
to
702df39
Compare
Closing in favor of smaller PRs, since migration needs it. Code removal now at #3701 |
THREESCALE-6549: Remove all traces of 'authorize.net' from the code and the DB
QUESTION: I needed to restart the server for Liquid to reload tags. What will happen in production?
TODOs