Better warning message about users with missing emails/principal names

solidify · Sep 29, 2023 · 1995137 · 1995137
1 parent e023e82
commit 1995137
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/src/WorkItemMigrator/JiraExport/JiraProvider.cs b/src/WorkItemMigrator/JiraExport/JiraProvider.cs
@@ -375,8 +375,15 @@ public string GetUserEmail(string usernameOrAccountId)
                 {
                     Logger.Log(LogLevel.Warning,
                         Settings.UsingJiraCloud
-                            ? $"Email is not public for user '{usernameOrAccountId}' in Jira, using usernameOrAccountId '{usernameOrAccountId}' for mapping."
-                            : $"Email for user '{usernameOrAccountId}' not found in Jira, using username '{usernameOrAccountId}' for mapping.");
+                            ? $"Email is not public for user '{usernameOrAccountId}' in Jira," +
+                            $" using usernameOrAccountId '{usernameOrAccountId}' for mapping." +
+                            $" You may safely ignore this warning, unless there is a subsequent warning about" +
+                            $" the username/accountId being missing in the usermapping file." +
+                            : $"Email for user '{usernameOrAccountId}' not found in Jira," +
+                            $" using username '{usernameOrAccountId}' for mapping." +
+                            $" You may safely ignore this warning, unless there is a subsequent warning about" +
+                            $" the username/accountId being missing in the usermapping file." +
+                    );
                 }
                 email = isUserEmailMissing ? usernameOrAccountId : user.Email;
                 _userEmailCache.Add(usernameOrAccountId, email);