From e92ba412d8574969cd22b637720e8d2bcd9ef610 Mon Sep 17 00:00:00 2001
From: Erez Rokah <erezrokah@users.noreply.github.com>
Date: Sun, 22 Mar 2020 16:04:11 +0200
Subject: [PATCH] chore: populate error boundry issue with relevant values
 (#3460)

---
 packages/netlify-cms-core/src/bootstrap.js    |  6 +-
 .../src/components/UI/ErrorBoundary.js        | 68 ++++++++++++++++++-
 2 files changed, 68 insertions(+), 6 deletions(-)

diff --git a/packages/netlify-cms-core/src/bootstrap.js b/packages/netlify-cms-core/src/bootstrap.js
index 4a9ffe9b..cc9d3741 100644
--- a/packages/netlify-cms-core/src/bootstrap.js
+++ b/packages/netlify-cms-core/src/bootstrap.js
@@ -18,10 +18,10 @@ import 'what-input';
 
 const ROOT_ID = 'nc-root';
 
-const TranslatedApp = ({ locale }) => {
+const TranslatedApp = ({ locale, config }) => {
   return (
     <I18n locale={locale} messages={getPhrases(locale)}>
-      <ErrorBoundary showBackup>
+      <ErrorBoundary showBackup config={config}>
         <ConnectedRouter history={history}>
           <Route component={App} />
         </ConnectedRouter>
@@ -31,7 +31,7 @@ const TranslatedApp = ({ locale }) => {
 };
 
 const mapDispatchToProps = state => {
-  return { locale: selectLocale(state.config) };
+  return { locale: selectLocale(state.config), config: state.config };
 };
 
 const ConnectedTranslatedApp = connect(mapDispatchToProps)(TranslatedApp);
diff --git a/packages/netlify-cms-core/src/components/UI/ErrorBoundary.js b/packages/netlify-cms-core/src/components/UI/ErrorBoundary.js
index fb04339b..884aa160 100644
--- a/packages/netlify-cms-core/src/components/UI/ErrorBoundary.js
+++ b/packages/netlify-cms-core/src/components/UI/ErrorBoundary.js
@@ -1,12 +1,69 @@
 import React from 'react';
 import PropTypes from 'prop-types';
+import ImmutablePropTypes from 'react-immutable-proptypes';
 import { translate } from 'react-polyglot';
 import styled from '@emotion/styled';
+import yaml from 'js-yaml';
+import { truncate } from 'lodash';
 import copyToClipboard from 'copy-text-to-clipboard';
 import { localForage } from 'netlify-cms-lib-util';
 import { buttons, colors } from 'netlify-cms-ui-default';
 
-const ISSUE_URL = 'https://github.com/netlify/netlify-cms/issues/new?template=bug_report.md';
+const ISSUE_URL = 'https://github.com/netlify/netlify-cms/issues/new?';
+const getIssueTemplate = ({ version, provider, browser, config }) => `
+**Describe the bug**
+
+**To Reproduce**
+
+**Expected behavior**
+
+**Screenshots**
+
+**Applicable Versions:**
+ - Netlify CMS version: \`${version}\`
+ - Git provider: \`${provider}\`
+ - Browser version: \`${browser}\`
+
+**CMS configuration**
+\`\`\`
+${config}
+\`\`\`
+
+**Additional context**
+`;
+
+const buildIssueTemplate = ({ config }) => {
+  let version = '';
+  if (typeof NETLIFY_CMS_VERSION === 'string') {
+    version = `netlify-cms@${NETLIFY_CMS_VERSION}`;
+  } else if (typeof NETLIFY_CMS_APP_VERSION === 'string') {
+    version = `netlify-cms-app@${NETLIFY_CMS_APP_VERSION}`;
+  }
+  const template = getIssueTemplate({
+    version,
+    provider: config.getIn(['backend', 'name']),
+    browser: navigator.userAgent,
+    config: yaml.safeDump(config.toJS()),
+  });
+
+  return template;
+};
+
+const buildIssueUrl = ({ title, config }) => {
+  try {
+    const body = buildIssueTemplate({ config });
+
+    const params = new URLSearchParams();
+    params.append('title', truncate(title, { length: 100 }));
+    params.append('body', truncate(body, { length: 4000, omission: '\n...' }));
+    params.append('labels', 'type: bug');
+
+    return `${ISSUE_URL}${params.toString()}`;
+  } catch (e) {
+    console.log(e);
+    return `${ISSUE_URL}template=bug_report.md`;
+  }
+};
 
 const ErrorBoundaryContainer = styled.div`
   padding: 40px;
@@ -67,6 +124,7 @@ class ErrorBoundary extends React.Component {
   static propTypes = {
     children: PropTypes.node,
     t: PropTypes.func.isRequired,
+    config: ImmutablePropTypes.map.isRequired,
   };
 
   state = {
@@ -92,7 +150,7 @@ class ErrorBoundary extends React.Component {
   async componentDidUpdate() {
     if (this.props.showBackup) {
       const backup = await localForage.getItem('backup');
-      console.log(backup);
+      backup && console.log(backup);
       this.setState({ backup });
     }
   }
@@ -108,7 +166,11 @@ class ErrorBoundary extends React.Component {
         <h1>{t('ui.errorBoundary.title')}</h1>
         <p>
           <span>{t('ui.errorBoundary.details')}</span>
-          <a href={ISSUE_URL} target="_blank" rel="noopener noreferrer">
+          <a
+            href={buildIssueUrl({ title: errorMessage, config: this.props.config })}
+            target="_blank"
+            rel="noopener noreferrer"
+          >
             {t('ui.errorBoundary.reportIt')}
           </a>
         </p>