From ca8b727ff520fb0641c1a532dedcf3506e26ebdc Mon Sep 17 00:00:00 2001
From: Erik Engervall <erik.engervall@gmail.com>
Date: Thu, 6 May 2021 14:47:56 +0200
Subject: [PATCH] Make console.log optional and accessible via a button

Signed-off-by: Erik Engervall <erik.engervall@gmail.com>
---
 .../src/features/Stats/DialogBody.tsx         | 19 +-----
 .../src/features/Stats/Warn.tsx               | 63 ++++++++++++-------
 2 files changed, 44 insertions(+), 38 deletions(-)

diff --git a/plugins/git-release-manager/src/features/Stats/DialogBody.tsx b/plugins/git-release-manager/src/features/Stats/DialogBody.tsx
index 04c64b7b07..219bc2a146 100644
--- a/plugins/git-release-manager/src/features/Stats/DialogBody.tsx
+++ b/plugins/git-release-manager/src/features/Stats/DialogBody.tsx
@@ -17,7 +17,6 @@
 import React from 'react';
 import { Alert } from '@material-ui/lab';
 import {
-  Box,
   makeStyles,
   Table,
   TableBody,
@@ -70,20 +69,6 @@ export function DialogBody() {
     project,
   });
 
-  const shouldWarn =
-    releaseStats.unmappableTags.length > 0 ||
-    releaseStats.unmatchedTags.length > 0 ||
-    releaseStats.unmatchedReleases.length > 0;
-
-  if (shouldWarn) {
-    // eslint-disable-next-line no-console
-    console.log("⚠️ Here's a summary of unmapped/unmatched tags/releases", {
-      unmatchedReleases: releaseStats.unmatchedReleases,
-      unmatchedTags: releaseStats.unmatchedTags,
-      unmappableTags: releaseStats.unmappableTags,
-    });
-  }
-
   return (
     <ReleaseStatsContext.Provider value={{ releaseStats }}>
       <Info />
@@ -115,7 +100,9 @@ export function DialogBody() {
           </TableBody>
         </Table>
 
-        <Box marginTop={2}>{shouldWarn && <Warn />}</Box>
+        {(releaseStats.unmappableTags.length > 0 ||
+          releaseStats.unmatchedTags.length > 0 ||
+          releaseStats.unmatchedReleases.length > 0) && <Warn />}
       </TableContainer>
     </ReleaseStatsContext.Provider>
   );
diff --git a/plugins/git-release-manager/src/features/Stats/Warn.tsx b/plugins/git-release-manager/src/features/Stats/Warn.tsx
index 93c2cc4154..24997cc9b3 100644
--- a/plugins/git-release-manager/src/features/Stats/Warn.tsx
+++ b/plugins/git-release-manager/src/features/Stats/Warn.tsx
@@ -16,6 +16,7 @@
 
 import React from 'react';
 import { Alert } from '@material-ui/lab';
+import { Box, Button } from '@material-ui/core';
 
 import { useProjectContext } from '../../contexts/ProjectContext';
 import { useReleaseStatsContext } from './contexts/ReleaseStatsContext';
@@ -25,30 +26,48 @@ export const Warn = () => {
   const { project } = useProjectContext();
 
   return (
-    <Alert severity="warning" style={{ marginBottom: 10 }}>
-      {releaseStats.unmappableTags.length > 0 && (
-        <div>
-          Failed to map <strong>{releaseStats.unmappableTags.length}</strong>{' '}
-          tags to releases
-        </div>
-      )}
+    <Box marginTop={2}>
+      <Alert severity="warning" style={{ marginBottom: 10 }}>
+        {releaseStats.unmappableTags.length > 0 && (
+          <div>
+            Failed to map <strong>{releaseStats.unmappableTags.length}</strong>{' '}
+            tags to releases
+          </div>
+        )}
 
-      {releaseStats.unmatchedTags.length > 0 && (
-        <div>
-          Failed to match <strong>{releaseStats.unmatchedTags.length}</strong>{' '}
-          tags to {project.versioningStrategy}
-        </div>
-      )}
+        {releaseStats.unmatchedTags.length > 0 && (
+          <div>
+            Failed to match <strong>{releaseStats.unmatchedTags.length}</strong>{' '}
+            tags to {project.versioningStrategy}
+          </div>
+        )}
 
-      {releaseStats.unmatchedReleases.length > 0 && (
-        <div>
-          Failed to match{' '}
-          <strong>{releaseStats.unmatchedReleases.length}</strong> releases to{' '}
-          {project.versioningStrategy}
-        </div>
-      )}
+        {releaseStats.unmatchedReleases.length > 0 && (
+          <div>
+            Failed to match{' '}
+            <strong>{releaseStats.unmatchedReleases.length}</strong> releases to{' '}
+            {project.versioningStrategy}
+          </div>
+        )}
 
-      <div>See full output in the console</div>
-    </Alert>
+        <Box marginTop={1} marginBottom={1}>
+          <Button
+            variant="contained"
+            color="secondary"
+            size="small"
+            onClick={() => {
+              // eslint-disable-next-line no-console
+              console.log("Here's all unmapped/unmatched tags/releases", {
+                unmatchedReleases: releaseStats.unmatchedReleases,
+                unmatchedTags: releaseStats.unmatchedTags,
+                unmappableTags: releaseStats.unmappableTags,
+              });
+            }}
+          >
+            Log all unmapped/unmatched tags/releases to the console
+          </Button>
+        </Box>
+      </Alert>
+    </Box>
   );
 };