More stripe integration #6649

Closed
mayeaux wants to merge 17 commits from more-stripe-integration into master
Showing only changes of commit b11caa82a3 - Show all commits

View file

@ -188,10 +188,15 @@ class SettingsStripeCard extends React.Component<Props, State> {
// instantiate stripe elements // instantiate stripe elements
setupStripe(); setupStripe();
}); });
// 500 error from the backend being down
} else if (error === 'internal_apis_down') { } else if (error === 'internal_apis_down') {
var displayString = 'There was an error from the server, please let support know'; var displayString = 'There was an error from the server, please let support know';
doToast({ message: displayString, isError: true }); doToast({ message: displayString, isError: true });
} else { } else {
// probably an error from stripe
var displayString = 'There was an error getting your card setup, please let support know';
jessopb commented 2021-07-29 22:59:34 +02:00 (Migrated from github.com)
Review

var -> let
Errors could probably be consts at the top?
API_DOWN_ERROR_MESSAGE
CARD_ERROR_MESSAGE

var -> let Errors could probably be consts at the top? API_DOWN_ERROR_MESSAGE CARD_ERROR_MESSAGE
doToast({ message: displayString, isError: true });
console.log('Unseen before error'); console.log('Unseen before error');
jessopb commented 2021-07-29 22:59:40 +02:00 (Migrated from github.com)
Review

log?

log?
} }
}); });