Skip to content

chore: remove Checkout.com integration#3702

Closed
TaprootFreak wants to merge 2 commits into
developfrom
feat/remove-checkout-integration
Closed

chore: remove Checkout.com integration#3702
TaprootFreak wants to merge 2 commits into
developfrom
feat/remove-checkout-integration

Conversation

@TaprootFreak
Copy link
Copy Markdown
Collaborator

Summary

Closes #3699. Removes the Checkout.com payment integration entirely:

  • Deletes src/integration/checkout/ (SDK wrapper, module, dto, service) and the CheckoutObserver.
  • Deletes the CheckoutTx entity, service, repository, mock and the FiatPayIn module (its only purpose was Checkout sync).
  • Drops the four live consumers of CheckoutService: createPaymentLink in BuyService, refundPayment in BuyCryptoService, getBalances in BankAdapter, and the FiatPayInSyncService itself.
  • Removes the checkoutTx relation from BuyCrypto, Transaction and Recall, plus all entity.checkoutTx branches in AML, support, gs, history mappers and transaction services.
  • Drops the last7dCheckoutVolume parameter chain through amlCheckAndFillUp / getAmlResult / getAmlErrors / amlRuleCheck.
  • Removes the dead AmlRule quote-check branches that fired only on paymentMethodIn === CARD; paymentMethodIn is no longer needed in amlRuleQuoteCheck.
  • Strips the checkout config block, CKO_* env vars, the bicep params (ckoPublicKey/ckoSecretKey/ckoEntityId) and the checkout-sdk-node dependency.
  • Cleans up corresponding test specs and the unused paymentLink field on BuyPaymentInfoDto / CustodyOrderBuyResponseDto.

Preserved on purpose

  • Existing *Checkout* migrations stay untouched (historical record).
  • AmlRule enum values 1/4/6/7/9/10/12/13 stay (DB-referenced); their cases simply have no live branch anymore.
  • TransactionTypeInternal.CHECKOUT_LTD, Blockchain.CHECKOUT, CardBankName.CHECKOUT, TransactionRequest.paymentLink and the dormant checkout_tx table remain to keep historical rows readable. A follow-up cleanup migration can be filed separately.

63 files changed, +81 / -1414.

Test plan

  • yarn lint
  • yarn format:check
  • yarn build
  • yarn test (931 passed)
  • Smoke check in DEV after deploy that BuyService/BuyCrypto/AML/Support flows still work end-to-end.

Checkout.com is no longer used as a payment provider. Drops the SDK,
service, observer, dto, the CheckoutTx entity/service/repo and the
FiatPayIn module. Removes all consumers (createPaymentLink, refundPayment,
getBalances, getPayments sync) along with checkoutTx relations on
BuyCrypto/Transaction/Recall, the related AML branches and the
last7dCheckoutVolume parameter chain.

Existing checkout-related migrations and the AmlRule enum values are
preserved for DB compatibility.

Closes #3699
Comment thread docs/david-analysis/01_extract_local.py Fixed
Comment thread docs/david-analysis/01_extract_local.py Fixed
Comment thread docs/david-analysis/01_extract_local.py Fixed
Comment thread docs/david-analysis/01_extract_local.py Fixed
Comment thread docs/david-analysis/01_extract_local.py Fixed
Comment thread scripts/generate-email-previews.js Fixed
Comment thread scripts/generate-email-previews.js Fixed
Comment thread scripts/generate-email-previews.js Fixed
Removes files unrelated to the Checkout.com removal that were swept in
by a wide `git add -A` (untracked local docs and scripts).
@TaprootFreak
Copy link
Copy Markdown
Collaborator Author

Superseded by #3703 (same diff, single commit).

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

Successfully merging this pull request may close these issues.

Remove Checkout.com integration (no longer in use)

1 participant