remove getfeedback (#1947)

* remove getfeedback

* remove GF_SURVEY_KEY

---------

Co-authored-by: Alexandra Carrillo <[email protected]>

Author: AndyMBridges

.github/workflows/build.yml

@@ -10,14 +10,11 @@ on:
         required: true
       SENTRY_KEY:
         required: true
-      GF_SURVEY_KEY:
-        required: true
 
 env:
   SEGMENT_ANALYTICS_KEY: ${{ secrets.SEGMENT_ANALYTICS_KEY }}
   LD_CLIENT_ID: ${{ secrets.LD_CLIENT_ID }}
   SENTRY_KEY: ${{ secrets.SENTRY_KEY }}
-  GF_SURVEY_KEY: ${{ secrets.GF_SURVEY_KEY }}
 
 jobs:
   build:

.github/workflows/case.yml

@@ -10,22 +10,19 @@ on:
         required: true
       SENTRY_KEY:
         required: true
-      GF_SURVEY_KEY:
-        required: true
 
 env:
   SEGMENT_ANALYTICS_KEY: ${{ secrets.SEGMENT_ANALYTICS_KEY }}
   LD_CLIENT_ID: ${{ secrets.LD_CLIENT_ID }}
   SENTRY_KEY: ${{ secrets.SENTRY_KEY }}
-  GF_SURVEY_KEY: ${{ secrets.GF_SURVEY_KEY }}
 
 jobs:
   case:
     name: Check for case being inconsistent
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        folder: ["docs"]
+        folder: ['docs']
     permissions:
       contents: read
     steps:

.github/workflows/lint.yml

@@ -10,14 +10,11 @@ on:
         required: true
       SENTRY_KEY:
         required: true
-      GF_SURVEY_KEY:
-        required: true
 
 env:
   SEGMENT_ANALYTICS_KEY: ${{ secrets.SEGMENT_ANALYTICS_KEY }}
   LD_CLIENT_ID: ${{ secrets.LD_CLIENT_ID }}
   SENTRY_KEY: ${{ secrets.SENTRY_KEY }}
-  GF_SURVEY_KEY: ${{ secrets.GF_SURVEY_KEY }}
 
 jobs:
   lint:
@@ -36,20 +33,20 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        file-extensions: [".md", ".mdx"]
+        file-extensions: ['.md', '.mdx']
     steps:
       - uses: actions/checkout@v4
       - name: Vale
         uses: Consensys/docs-gha/spelling@main
         with:
-          FILEPATHS: "snaps,services,wallet,developer-tools,docs"
+          FILEPATHS: 'snaps,services,wallet,developer-tools,docs'
 
   linkCheck:
     name: Link Checking
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        file-extensions: [".md", ".mdx"]
+        file-extensions: ['.md', '.mdx']
     steps:
       - uses: actions/checkout@v4
       - name: LinkCheck

.github/workflows/main.yml

@@ -26,7 +26,6 @@ jobs:
       SEGMENT_ANALYTICS_KEY: ${{ secrets.SEGMENT_ANALYTICS_DEV_KEY }}
       LD_CLIENT_ID: ${{ secrets.LD_CLIENT_ID }}
       SENTRY_KEY: ${{ secrets.SENTRY_KEY }}
-      GF_SURVEY_KEY: ${{ secrets.GF_SURVEY_KEY }}
 
   lint:
     name: Lint
@@ -35,7 +34,6 @@ jobs:
       SEGMENT_ANALYTICS_KEY: ${{ secrets.SEGMENT_ANALYTICS_DEV_KEY }}
       LD_CLIENT_ID: ${{ secrets.LD_CLIENT_ID }}
       SENTRY_KEY: ${{ secrets.SENTRY_KEY }}
-      GF_SURVEY_KEY: ${{ secrets.GF_SURVEY_KEY }}
 
   case:
     name: Case
@@ -44,7 +42,6 @@ jobs:
       SEGMENT_ANALYTICS_KEY: ${{ secrets.SEGMENT_ANALYTICS_DEV_KEY }}
       LD_CLIENT_ID: ${{ secrets.LD_CLIENT_ID }}
       SENTRY_KEY: ${{ secrets.SENTRY_KEY }}
-      GF_SURVEY_KEY: ${{ secrets.GF_SURVEY_KEY }}
 
   all-jobs-completed:
     name: All jobs completed

docusaurus.config.js

@@ -42,7 +42,6 @@ const config = {
     VERCEL_ENV: process.env.VERCEL_ENV,
     DASHBOARD_URL: process.env.DASHBOARD_URL || 'http://localhost:3000',
     SENTRY_KEY: process.env.SENTRY_KEY,
-    GF_SURVEY_KEY: process.env.GF_SURVEY_KEY,
     LINEA_ENS_URL: process.env.LINEA_ENS_URL,
   },
 
@@ -57,8 +56,6 @@ const config = {
       defer: true,
       'data-domain': 'docs.metamask.io',
     },
-    { src: '/js/feedback-script.js', defer: true, async: true },
-    { src: '/js/getfeedback.js', defer: true, async: true },
   ],
 
   markdown: {
@@ -296,16 +293,16 @@ const config = {
                 href: 'https://github.com/MetaMask/metamask-docs',
               },
               {
-                label: "MetaMask wallet GitHub",
-                href: "https://github.com/MetaMask/metamask-extension/",
+                label: 'MetaMask wallet GitHub',
+                href: 'https://github.com/MetaMask/metamask-extension/',
               },
               {
                 label: 'MetaMask SDK GitHub',
                 href: 'https://github.com/MetaMask/metamask-sdk/',
               },
               {
-                label: "MetaMask Mobile GitHub",
-                href: "https://github.com/MetaMask/metamask-mobile",
+                label: 'MetaMask Mobile GitHub',
+                href: 'https://github.com/MetaMask/metamask-mobile',
               },
               {
                 label: 'Snaps GitHub',

src/theme/DocItem/Layout/GetFeedback.jsx

@@ -1,55 +1,54 @@
-import React from "react";
-import clsx from "clsx";
-import { useWindowSize } from "@docusaurus/theme-common";
-import { useDoc } from '@docusaurus/plugin-content-docs/client';
-import DocItemPaginator from "@theme/DocItem/Paginator";
-import DocVersionBanner from "@theme/DocVersionBanner";
-import DocItemFooter from "@theme/DocItem/Footer";
-import DocItemTags from "@theme/DocItem/Tags";
-import DocItemTOCMobile from "@theme/DocItem/TOC/Mobile";
-import DocItemTOCDesktop from "@theme/DocItem/TOC/Desktop";
-import DocItemContent from "@theme/DocItem/Content";
-import DocBreadcrumbs from "@theme/DocBreadcrumbs";
-import styles from "./styles.module.css";
-import GetFeedback from "./GetFeedback.jsx";
+import React from 'react'
+import clsx from 'clsx'
+import { useWindowSize } from '@docusaurus/theme-common'
+import { useDoc } from '@docusaurus/plugin-content-docs/client'
+import DocItemPaginator from '@theme/DocItem/Paginator'
+import DocVersionBanner from '@theme/DocVersionBanner'
+import DocItemFooter from '@theme/DocItem/Footer'
+import DocItemTags from '@theme/DocItem/Tags'
+import DocItemTOCMobile from '@theme/DocItem/TOC/Mobile'
+import DocItemTOCDesktop from '@theme/DocItem/TOC/Desktop'
+import DocItemContent from '@theme/DocItem/Content'
+import DocBreadcrumbs from '@theme/DocBreadcrumbs'
+import styles from './styles.module.css'
 /**
  * Decide if the toc should be rendered, on mobile or desktop viewports
  */
 function useDocTOC() {
-  const { frontMatter, toc } = useDoc();
-  const windowSize = useWindowSize();
-  const hidden = frontMatter.hide_table_of_contents;
-  const canRender = !hidden && toc.length > 0;
-  const mobile = canRender ? <DocItemTOCMobile /> : undefined;
+  const { frontMatter, toc } = useDoc()
+  const windowSize = useWindowSize()
+  const hidden = frontMatter.hide_table_of_contents
+  const canRender = !hidden && toc.length > 0
+  const mobile = canRender ? <DocItemTOCMobile /> : undefined
   const desktop =
-    canRender && (windowSize === "desktop" || windowSize === "ssr") ? (
+    canRender && (windowSize === 'desktop' || windowSize === 'ssr') ? (
       <DocItemTOCDesktop />
-    ) : undefined;
+    ) : undefined
   return {
     hidden,
     mobile,
     desktop,
-  };
+  }
 }
 export default function DocItemLayout({ children }) {
-  const docTOC = useDocTOC();
+  const docTOC = useDocTOC()
   return (
     <div className="row">
-      <div className={clsx("col", !docTOC.hidden && styles.docItemCol)}>
+      <div className={clsx('col', !docTOC.hidden && styles.docItemCol)}>
         <DocVersionBanner />
         <div className={styles.docItemContainer}>
           <article>
             <DocBreadcrumbs />
             <DocItemTags />
             {docTOC.mobile}
             <DocItemContent>{children}</DocItemContent>
-            {/* <GetFeedback /> */}
+
             <DocItemFooter />
           </article>
           <DocItemPaginator />
         </div>
       </div>
       {docTOC.desktop && <div className="col col--3">{docTOC.desktop}</div>}
     </div>
-  );
+  )
 }