diff --git a/app/assets/javascripts/browsered/web-payments/apple-pay.js b/app/assets/javascripts/browsered/web-payments/apple-pay.js index 03daa7c0f..e39feb4b1 100644 --- a/app/assets/javascripts/browsered/web-payments/apple-pay.js +++ b/app/assets/javascripts/browsered/web-payments/apple-pay.js @@ -79,7 +79,6 @@ module.exports = () => { }).then(response => { if (response.status >= 200 && response.status < 300) { ga('send', 'event', 'Apple Pay', 'Successful', 'auth/capture request') - sendLogMessage(window.paymentDetails.chargeID, 'ApplePaySuccessfulAuthorisation') return response.json().then(data => { session.completePayment(ApplePaySession.STATUS_SUCCESS) window.location.href = data.url @@ -90,7 +89,6 @@ module.exports = () => { toggleSubmitButtons() showErrorSummary(i18n.fieldErrors.webPayments.apple) ga('send', 'event', 'Apple Pay', 'Error', 'During authorisation/capture') - sendLogMessage(window.paymentDetails.chargeID, 'ApplePayFailedAuthorisation') } }).catch(err => { session.abort() @@ -98,7 +96,6 @@ module.exports = () => { toggleSubmitButtons() showErrorSummary(i18n.fieldErrors.webPayments.apple) ga('send', 'event', 'Apple Pay', 'Error', 'Couldn’t post to /web-payments-auth-request/apple/{chargeId}') - sendLogMessage(window.paymentDetails.chargeID, 'ApplePayErrorMakingRequestToAuthorise') return err }) } diff --git a/app/assets/javascripts/browsered/web-payments/google-pay.js b/app/assets/javascripts/browsered/web-payments/google-pay.js index a8336d585..48202756b 100644 --- a/app/assets/javascripts/browsered/web-payments/google-pay.js +++ b/app/assets/javascripts/browsered/web-payments/google-pay.js @@ -29,7 +29,6 @@ const submitGooglePayAuthRequest = (paymentResponse) => { }) .then(response => { ga('send', 'event', 'Google Pay', 'Successful', 'auth/capture request') - sendLogMessage(window.paymentDetails.chargeID, 'GooglePaySuccessfulAuthorisation') if (response.status >= 200 && response.status < 300) { return response.json().then(data => { window.location.href = data.url @@ -39,7 +38,6 @@ const submitGooglePayAuthRequest = (paymentResponse) => { toggleSubmitButtons() showErrorSummary(i18n.fieldErrors.webPayments.failureTitle, i18n.fieldErrors.webPayments.failureBody) ga('send', 'event', 'Google Pay', 'Error', 'During authorisation/capture') - sendLogMessage(window.paymentDetails.chargeID, 'GooglePayFailedAuthorisation') } }) .catch(err => { @@ -47,7 +45,6 @@ const submitGooglePayAuthRequest = (paymentResponse) => { toggleSubmitButtons() showErrorSummary(i18n.fieldErrors.webPayments.failureTitle, i18n.fieldErrors.webPayments.failureBody) ga('send', 'event', 'Google Pay', 'Error', 'During authorisation/capture') - sendLogMessage(window.paymentDetails.chargeID, 'GooglePayErrorMakingRequestToAuthorise') return err }) } diff --git a/app/controllers/client-side-logging.controller.js b/app/controllers/client-side-logging.controller.js index bc4eaea81..a902a90c8 100644 --- a/app/controllers/client-side-logging.controller.js +++ b/app/controllers/client-side-logging.controller.js @@ -9,15 +9,9 @@ const LOG_CODES = { ApplePayAborted: 'Apple Pay attempt aborted by user', ApplePayMerchantIdNotValid: 'Apple Pay Merchant ID not valid', ApplePayMerchantValidationError: 'Error completing Apple Pay merchant validation', - ApplePayErrorMakingRequestToAuthorise: 'There was an error making a request to the server to authorise an Apple Pay payment', - ApplePaySuccessfulAuthorisation: 'Apple Pay authorisation was successful', - ApplePayFailedAuthorisation: 'Apple Pay authorisation failed', GooglePayAvailable: 'Google Pay is available on this device', GooglePayStarted: 'User chose Google Pay method', - GooglePayAborted: 'Google Pay attempt aborted by user', - GooglePayErrorMakingRequestToAuthorise: 'There was an error making a request to the server to authorise an Google Pay payment', - GooglePaySuccessfulAuthorisation: 'Google Pay authorisation was successful', - GooglePayFailedAuthorisation: 'Google Pay authorisation failed' + GooglePayAborted: 'Google Pay attempt aborted by user' } function log (req, res) {