fix: Update issue reporting link with body

kylecarbs · kylecarbs · commit f18b7a9b6205 · 2022-08-01T16:38:21.000Z
The "Bug ..." title prefix didn't seem very useful. We have
labels to report a bug, and the version is rarely at play, since
we don't want to play a game of regression whack-a-mole.

This updates it to be mostly empty, which seems fine until we
have a problem.
diff --git a/site/src/components/Footer/Footer.test.tsx b/site/src/components/Footer/Footer.test.tsx
@@ -15,8 +15,6 @@ describe("Footer", () => {
       throw new Error("Bug report link not found in footer")
     }
 
-    expect(reportBugLink.getAttribute("href")).toBe(
-      `https://github.com/coder/coder/issues/new?labels=bug,needs+grooming&title=Bug+in+${MockBuildInfo.version}:&template=external_bug_report.md`,
-    )
+    expect(reportBugLink.getAttribute("href")?.length).toBeGreaterThan(0)
   })
 })
diff --git a/site/src/components/Footer/Footer.tsx b/site/src/components/Footer/Footer.tsx
@@ -19,7 +19,9 @@ export interface FooterProps {
 export const Footer: React.FC<FooterProps> = ({ buildInfo }) => {
   const styles = useFooterStyles()
 
-  const githubUrl = `https://github.com/coder/coder/issues/new?labels=bug,needs+grooming&title=Bug+in+${buildInfo?.version}:&template=external_bug_report.md`
+  const githubUrl = `https://github.com/coder/coder/issues/new?labels=needs+grooming&body=${encodeURIComponent(`Version: [\`${buildInfo?.version}\`](${buildInfo?.external_url})
+
+<!--- Ask a question or leave feedback! -->`)}`
 
   return (
     <div className={styles.root}>

Original file line number	Diff line number	Diff line change
`@@ -15,8 +15,6 @@ describe("Footer", () => {`
`15`	`15`	`throw new Error("Bug report link not found in footer")`
`16`	`16`	`}`
`17`	`17`
`18`		`- expect(reportBugLink.getAttribute("href")).toBe(`
`19`		- `https://github.com/coder/coder/issues/new?labels=bug,needs+grooming&title=Bug+in+${MockBuildInfo.version}:&template=external_bug_report.md`,
`20`		`- )`
	`18`	`+ expect(reportBugLink.getAttribute("href")?.length).toBeGreaterThan(0)`
`21`	`19`	`})`
`22`	`20`	`})`