Skip to content

Commit

Permalink
Merge pull request #324 from Adyen/adjustment/removeStoredDeletionErr…
Browse files Browse the repository at this point in the history
…orMessage

Remove stored deletion error message
  • Loading branch information
Robert-SD authored Dec 20, 2024
2 parents adf1967 + 22082e0 commit e6716d4
Show file tree
Hide file tree
Showing 6 changed files with 4 additions and 37 deletions.
3 changes: 2 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,9 @@

* Added support for partial payments.
* Improved dynamic viewport of card component.
* Added support of iDEAL to the Instant Component.
* Added support of iDEAL to the Instant Component.
* Added the missing loading bottom sheet for the advanced flow google pay component.
* Removed alert in case the deletion of stored payment method failed.
* Updated iOS SDK to v5.14.0.
* Updated Android SDK to v5.8.0. Gradle v8 is now mandatory.

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -200,13 +200,7 @@ class AdvancedDropInService : DropInService(), LifecycleOwner {
return if (deleteStoredPaymentMethodResultDTO?.isSuccessfullyRemoved == true) {
RecurringDropInServiceResult.PaymentMethodRemoved(deleteStoredPaymentMethodResultDTO.storedPaymentMethodId)
} else {
// TODO - the error message should be provided by the native SDK
RecurringDropInServiceResult.Error(
errorDialog =
ErrorDialog(
message = "Removal of the stored payment method failed. Please try again later."
)
)
RecurringDropInServiceResult.Error(errorDialog = null)
}
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -50,13 +50,7 @@ class SessionDropInService : SessionDropInService(), LifecycleOwner {
return if (deleteStoredPaymentMethodResultDTO?.isSuccessfullyRemoved == true) {
RecurringDropInServiceResult.PaymentMethodRemoved(deleteStoredPaymentMethodResultDTO.storedPaymentMethodId)
} else {
// TODO - the error message should be provided by the native SDK
RecurringDropInServiceResult.Error(
errorDialog =
ErrorDialog(
message = "Removal of the stored payment method failed. Please try again later."
)
)
RecurringDropInServiceResult.Error(errorDialog = null)
}
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,11 +24,6 @@ class DropInAdvancedFlowStoredPaymentMethodsDelegate: StoredPaymentMethodsDelega
}

func handleDisableResult(isSuccessfullyRemoved: Bool) {
if isSuccessfullyRemoved == false {
let errorAlert = TemporaryAlertHelper.buildPaymentMethodDeletionErrorAlert()
viewController.adyen.topPresenter.present(errorAlert, animated: true)
}

completionHandler?(isSuccessfullyRemoved)
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -24,11 +24,6 @@ class DropInSessionsStoredPaymentMethodsDelegate: StoredPaymentMethodsDelegate {
}

func handleDisableResult(isSuccessfullyRemoved: Bool) {
if isSuccessfullyRemoved == false {
let errorAlert = TemporaryAlertHelper.buildPaymentMethodDeletionErrorAlert()
viewController.adyen.topPresenter.present(errorAlert, animated: true)
}

completionHandler?(isSuccessfullyRemoved)
}
}
12 changes: 0 additions & 12 deletions ios/Classes/utils/TemporaryAlertHelper.swift

This file was deleted.

0 comments on commit e6716d4

Please sign in to comment.