paulononato/backstage
1
0
Fork 0
Code Issues Pull Requests Actions 41 Packages Projects Releases Wiki Activity

Make console.log optional and accessible via a button

Browse Source 
Signed-off-by: Erik Engervall <erik.engervall@gmail.com>
This commit is contained in:
Erik Engervall
2021-05-06 14:47:56 +02:00
parent a8ae9a5234
commit ca8b727ff5
2 changed files with 44 additions and 38 deletions
Download Patch File Download Diff File Expand all files Collapse all files
plugins/git-release-manager/src/features/Stats/DialogBody.tsx
+3 -16
View File
@@ -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>
);
plugins/git-release-manager/src/features/Stats/Warn.tsx
+41 -22
View File
@@ -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>
);
};