Merge pull request #1398 from 0xProject/feature/instant/more-error-reporting
[instant] More aggressive error reporting
This commit is contained in:
@@ -38,14 +38,11 @@ export const buyQuoteUpdater = {
|
|||||||
} catch (error) {
|
} catch (error) {
|
||||||
const errorMessage = assetUtils.assetBuyerErrorMessage(asset, error);
|
const errorMessage = assetUtils.assetBuyerErrorMessage(asset, error);
|
||||||
|
|
||||||
if (_.isUndefined(errorMessage)) {
|
errorReporter.report(error);
|
||||||
// This is an unknown error, report it to rollbar
|
analytics.trackQuoteError(error.message ? error.message : 'other', baseUnitValue, fetchOrigin);
|
||||||
errorReporter.report(error);
|
|
||||||
}
|
|
||||||
|
|
||||||
if (options.dispatchErrors) {
|
if (options.dispatchErrors) {
|
||||||
dispatch(actions.setQuoteRequestStateFailure());
|
dispatch(actions.setQuoteRequestStateFailure());
|
||||||
analytics.trackQuoteError(error.message ? error.message : 'other', baseUnitValue, fetchOrigin);
|
|
||||||
errorFlasher.flashNewErrorMessage(dispatch, errorMessage || 'Error fetching price, please try again');
|
errorFlasher.flashNewErrorMessage(dispatch, errorMessage || 'Error fetching price, please try again');
|
||||||
}
|
}
|
||||||
return;
|
return;
|
||||||
|
Reference in New Issue
Block a user