From 54277eae28f42ee328a87c65dfdb79b062100a2b Mon Sep 17 00:00:00 2001 From: George Goldberg Date: Mon, 31 Oct 2016 13:02:54 +0000 Subject: [PATCH] Warning in log when a Slack channel import fails. (#4262) Previously it just wrote to the summary, and as a debug level message in the log, so if you had the log level set to warn, you might never find out that importing a channel failed. --- api/slackimport.go | 2 +- i18n/en.json | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/api/slackimport.go b/api/slackimport.go index b9dab98bae..af27b0d30e 100644 --- a/api/slackimport.go +++ b/api/slackimport.go @@ -312,7 +312,7 @@ func SlackAddChannels(teamId string, slackchannels []SlackChannel, posts map[str if mChannel == nil { // Maybe it already exists? if result := <-Srv.Store.Channel().GetByName(teamId, sChannel.Name); result.Err != nil { - l4g.Debug(utils.T("api.slackimport.slack_add_channels.import_failed.debug"), newChannel.DisplayName) + l4g.Warn(utils.T("api.slackimport.slack_add_channels.import_failed.warn"), newChannel.DisplayName) log.WriteString(utils.T("api.slackimport.slack_add_channels.import_failed", map[string]interface{}{"DisplayName": newChannel.DisplayName})) continue } else { diff --git a/i18n/en.json b/i18n/en.json index e6b4443707..a7e17cdf5e 100644 --- a/i18n/en.json +++ b/i18n/en.json @@ -1532,8 +1532,8 @@ "translation": "Failed to import: {{.DisplayName}}\r\n" }, { - "id": "api.slackimport.slack_add_channels.import_failed.debug", - "translation": "Failed to import: %s" + "id": "api.slackimport.slack_add_channels.import_failed.warn", + "translation": "Slack Importer: Failed to import channel: %s" }, { "id": "api.slackimport.slack_add_channels.merge",