Skip to content

Commit f3d7b50

Browse files
authored
Merge pull request #901 from Particular/missing-icon
Show warning icon for successful retried messages
2 parents 28271b2 + 44aa96e commit f3d7b50

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/ServiceInsight/MessageList/MessageStatusIconInfo.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ static MessageStatusIconInfo()
2222
[MessageStatus.Failed] = "Failed",
2323
[MessageStatus.ArchivedFailure] = "Archived",
2424
[MessageStatus.RepeatedFailure] = "RepeatedFailed",
25-
[MessageStatus.ResolvedSuccessfully] = "ResolvedSuccess",
25+
[MessageStatus.ResolvedSuccessfully] = "Successful",
2626
[MessageStatus.RetryIssued] = "RetryIssued",
2727
};
2828
}
@@ -42,7 +42,7 @@ public MessageStatusIconInfo(StoredMessage message, Func<string, ImageSource> re
4242
Image = GetImage();
4343
}
4444

45-
private bool Warn(StoredMessage message)
45+
private static bool Warn(StoredMessage message)
4646
{
4747
return message.ProcessingTime < TimeSpan.Zero ||
4848
message.CriticalTime < TimeSpan.Zero ||
@@ -70,11 +70,11 @@ ImageSource GetImage()
7070
var currentStatus = statusSpecified ? Status : MessageStatus.Successful;
7171
var imageName = $"MessageStatus_{statusToIconNameMap[currentStatus]}";
7272

73-
if (HasWarn)
73+
if (HasWarn || Status == MessageStatus.ResolvedSuccessfully)
7474
{
7575
imageName += "_Warn";
7676
}
77-
77+
7878
var image = resourceFinder(imageName);
7979

8080
return image;

0 commit comments

Comments
 (0)