fix: pass error message #2253

Merged
tzarebczan merged 2 commits from fix-download-error into master 2019-02-01 18:10:17 +01:00
3 changed files with 8 additions and 3 deletions

View file

@ -23,6 +23,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
- Flow errors ([#2213](https://github.com/lbryio/lbry-desktop/pull/2213))
- Video position on previously viewed files ([#2240](https://github.com/lbryio/lbry-desktop/pull/2240))
- Pass download error details on modal ([#2255](https://github.com/lbryio/lbry-desktop/pull/2255))
## [0.27.1] - 2018-01-22

View file

@ -180,12 +180,13 @@ function handleLoadVideoError(uri, errorType = '') {
data: { uri },
});
dispatch(doSetPlayingUri(null));
// this is not working, but should be it's own separate modal in the future (https://github.com/lbryio/lbry-desktop/issues/892)
if (errorType === 'timeout') {
doOpenModal(MODALS.FILE_TIMEOUT, { uri });
} else {
dispatch(
doError(
`Failed to download ${uri}, please try again. If this problem persists, visit https://lbry.io/faq/support for support.`
`Failed to download ${uri}, please try again or see error details:\n\n${errorType}\n\nIf this problem persists, visit https://lbry.io/support for help. `
)
);
}
@ -204,6 +205,7 @@ export function doLoadVideo(uri, shouldRecordViewEvent) {
Lbry.get({ uri })
.then(streamInfo => {
// need error code from SDK to capture properly
const timeout =
streamInfo === null || typeof streamInfo !== 'object' || streamInfo.error === 'Timeout';
@ -221,8 +223,8 @@ export function doLoadVideo(uri, shouldRecordViewEvent) {
}
}
})
.catch(() => {
dispatch(handleLoadVideoError(uri));
.catch(error => {
dispatch(handleLoadVideoError(uri, error));
});
};
}

View file

@ -107,6 +107,7 @@
.error-modal {
max-width: none;
width: var(--modal-width);
white-space: pre-wrap;
}
.error-modal__error-list {
@ -120,4 +121,5 @@
color: $lbry-red-5;
list-style: none;
overflow-y: scroll;
white-space: pre-wrap;
}