New: Improve messaging if release is in queue because all episodes in release were not imported
This commit is contained in:
parent
cd28af98ee
commit
2728bf79ca
|
@ -3,3 +3,7 @@
|
||||||
|
|
||||||
width: 30px;
|
width: 30px;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
.noMessages {
|
||||||
|
margin-bottom: 10px;
|
||||||
|
}
|
||||||
|
|
|
@ -12,7 +12,10 @@ function getDetailedPopoverBody(statusMessages) {
|
||||||
{
|
{
|
||||||
statusMessages.map(({ title, messages }) => {
|
statusMessages.map(({ title, messages }) => {
|
||||||
return (
|
return (
|
||||||
<div key={title}>
|
<div
|
||||||
|
key={title}
|
||||||
|
className={messages.length ? undefined: styles.noMessages}
|
||||||
|
>
|
||||||
{title}
|
{title}
|
||||||
<ul>
|
<ul>
|
||||||
{
|
{
|
||||||
|
|
|
@ -63,6 +63,8 @@ export const defaultState = {
|
||||||
};
|
};
|
||||||
|
|
||||||
export const persistState = [
|
export const persistState = [
|
||||||
|
'interactiveImport.sortKey',
|
||||||
|
'interactiveImport.sortDirection',
|
||||||
'interactiveImport.recentFolders',
|
'interactiveImport.recentFolders',
|
||||||
'interactiveImport.importMode'
|
'interactiveImport.importMode'
|
||||||
];
|
];
|
||||||
|
|
|
@ -126,16 +126,30 @@ namespace NzbDrone.Core.Download
|
||||||
if (importResults.Empty())
|
if (importResults.Empty())
|
||||||
{
|
{
|
||||||
trackedDownload.Warn("No files found are eligible for import in {0}", outputPath);
|
trackedDownload.Warn("No files found are eligible for import in {0}", outputPath);
|
||||||
|
|
||||||
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
var statusMessages = new List<TrackedDownloadStatusMessage>
|
||||||
|
{
|
||||||
|
new TrackedDownloadStatusMessage("One or more episodes expected in this release were not imported or missing", new List<string>())
|
||||||
|
};
|
||||||
|
|
||||||
if (importResults.Any(c => c.Result != ImportResultType.Imported))
|
if (importResults.Any(c => c.Result != ImportResultType.Imported))
|
||||||
{
|
{
|
||||||
var statusMessages = importResults
|
statusMessages.AddRange(
|
||||||
|
importResults
|
||||||
.Where(v => v.Result != ImportResultType.Imported && v.ImportDecision.LocalEpisode != null)
|
.Where(v => v.Result != ImportResultType.Imported && v.ImportDecision.LocalEpisode != null)
|
||||||
.Select(v => new TrackedDownloadStatusMessage(Path.GetFileName(v.ImportDecision.LocalEpisode.Path), v.Errors))
|
.OrderBy(v => v.ImportDecision.LocalEpisode.Path)
|
||||||
.ToArray();
|
.Select(v =>
|
||||||
|
new TrackedDownloadStatusMessage(Path.GetFileName(v.ImportDecision.LocalEpisode.Path),
|
||||||
|
v.Errors))
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
trackedDownload.Warn(statusMessages);
|
if (statusMessages.Any())
|
||||||
|
{
|
||||||
|
trackedDownload.Warn(statusMessages.ToArray());
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue