diff --git a/src/Umbraco.Web/HealthCheck/HealthCheckResults.cs b/src/Umbraco.Web/HealthCheck/HealthCheckResults.cs index 6c46e98876..fb36a8bc12 100644 --- a/src/Umbraco.Web/HealthCheck/HealthCheckResults.cs +++ b/src/Umbraco.Web/HealthCheck/HealthCheckResults.cs @@ -68,13 +68,20 @@ namespace Umbraco.Web.HealthCheck var checkName = result.Key; var checkResults = result.Value; var checkIsSuccess = result.Value.All(x => x.ResultType == StatusResultType.Success); + + // add a new line if not the first check + if (result.Equals(_results.First()) == false) + { + sb.Append(Environment.NewLine); + } + if (checkIsSuccess) { - sb.AppendFormat("{0}Checks for'{1}' all completed succesfully.{2}", newItem, checkName, Environment.NewLine); + sb.AppendFormat("{0}Checks for '{1}' all completed succesfully.{2}", newItem, checkName, Environment.NewLine); } else { - sb.AppendFormat("{0}Checks for'{1}' completed with errors.{2}", newItem, checkName, Environment.NewLine); + sb.AppendFormat("{0}Checks for '{1}' completed with errors.{2}", newItem, checkName, Environment.NewLine); } foreach (var checkResult in checkResults) diff --git a/src/Umbraco.Web/Scheduling/HealthCheckNotifier.cs b/src/Umbraco.Web/Scheduling/HealthCheckNotifier.cs index e20fc3d6e0..78fe5a8b44 100644 --- a/src/Umbraco.Web/Scheduling/HealthCheckNotifier.cs +++ b/src/Umbraco.Web/Scheduling/HealthCheckNotifier.cs @@ -89,10 +89,10 @@ namespace Umbraco.Web.Scheduling { var slackClient = new SlackClient(healthCheckConfig.NotificationSettings.SlackSettings.WebHookUrl); - var icon = Emoji.MinusOne; + var icon = Emoji.Warning; if (results.AllChecksSuccessful) { - icon = Emoji.PlusOne; + icon = Emoji.WhiteCheckMark; } var slackMessage = new SlackMessage