Chore: remove hotjar, as we're no longer using it

stride-nyc · Apr 5, 2022 · 0db9b26 · 0db9b26
1 parent 3385b0d
commit 0db9b26
Show file tree

Hide file tree

Showing 8 changed files with 3 additions and 40 deletions.
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -11,7 +11,6 @@ module.exports = {
     "expect": false,
     "beforeEach": false,
     "afterEach": false,
-    "hj": false,
     "Honeybadger": false,
     "mountWithConnectedSubcomponents": false,
   },

diff --git a/lib/remote_retro_web/templates/layout/app.html.eex b/lib/remote_retro_web/templates/layout/app.html.eex
@@ -13,16 +13,6 @@
       </script>
 
       <script>
-        // Hotjar Tracking Code for https://remoteretro.org
-        (function(h,o,t,j,a,r){
-          h.hj=h.hj||function(){(h.hj.q=h.hj.q||[]).push(arguments)};
-          h._hjSettings={hjid:650812,hjsv:6};
-          a=o.getElementsByTagName('head')[0];
-          r=o.createElement('script');r.async=1;
-          r.src=t+h._hjSettings.hjid+j+h._hjSettings.hjsv;
-          a.appendChild(r);
-        })(window,document,'https://static.hotjar.com/c/hotjar-','.js?sv=');
-
         // disable react dev tools in prod to eliminate possible attack vector
         var hasReactDevToolsExtension = typeof window.__REACT_DEVTOOLS_GLOBAL_HOOK__ === 'object'
         if (hasReactDevToolsExtension) {
@@ -35,7 +25,6 @@
     <% else %>
       <!-- stub out vendor script funcs in dev/test -->
       <script>
-        window.hj = () => {};
         window.Honeybadger = {
           notify: () => {},
           setContext: () => {},

diff --git a/test/components/remote_retro_test.js b/test/components/remote_retro_test.js
@@ -33,18 +33,6 @@ describe("RemoteRetro component", () => {
     },
   }
 
-  context("when the component mounts", () => {
-    it("triggers a hotjar event, passing the stage", () => {
-      const hotjarSpy = spy(global, "hj")
-
-      shallow(
-        <RemoteRetro {...defaultProps} stage={CLOSED} />
-      )
-
-      expect(hotjarSpy).calledWith("trigger", CLOSED)
-    })
-  })
-
   // we can't afford to have this integration break, as the grouping stage relies
   // on viewport manipulation
   it("renders a ViewportMetaTag, passing stage, alert, and browser orientation", () => {

diff --git a/test/support/js/test_helper.js b/test/support/js/test_helper.js
@@ -7,7 +7,6 @@ import Enzyme from "enzyme"
 import Adapter from "enzyme-adapter-react-16"
 import sinon from "sinon"
 import PropTypes from "prop-types"
-import noop from "lodash/noop"
 import STAGES from "../../../web/static/js/configs/stages"
 
 chai.use(chaiUUID)
@@ -18,7 +17,6 @@ const { IDEA_GENERATION } = STAGES
 
 Enzyme.configure({ adapter: new Adapter() })
 
-global.hj = noop
 global.expect = expect
 global.Image = function Image() {}
 global.Honeybadger = { notify: () => {} }

diff --git a/web/static/js/components/idea_content_base.jsx b/web/static/js/components/idea_content_base.jsx
@@ -26,7 +26,7 @@ const IdeaContentBase = props => {
         canUserEditIdeaContents={canUserEditIdeaContents}
       />
       <div className="text">
-        <span data-hj-masked>{ idea.body }</span>
+        <span>{ idea.body }</span>
         {assignee && <span className={styles.assignee}> ({assignee.name})</span>}
         {isEdited && <span className={styles.editedIndicator}> (edited)</span>}
       </div>

diff --git a/web/static/js/components/idea_live_edit_content.jsx b/web/static/js/components/idea_live_edit_content.jsx
@@ -8,7 +8,7 @@ const IdeaLiveEditContent = props => {
   return (
     <div className="ui raised segment">
       <p className="ui center aligned sub dividing header">Facilitator is Editing</p>
-      <span data-hj-masked>{idea.liveEditText}</span>
+      <span>{idea.liveEditText}</span>
     </div>
   )
 }

diff --git a/web/static/js/components/remote_retro.jsx b/web/static/js/components/remote_retro.jsx
@@ -14,17 +14,6 @@ import Error from "./error"
 import StageHelp from "./stage_help"
 
 export class RemoteRetro extends Component {
-  // Trigger analytics events on page load and stage changes
-  componentDidMount() {
-    const { stage } = this.props
-    hj("trigger", stage)
-  }
-
-  componentDidUpdate(prevProps) {
-    const { stage } = this.props
-    if (prevProps.stage !== stage) { hj("trigger", stage) }
-  }
-
   componentDidCatch(error) {
     const { stage, presences, usersById } = this.props
 

diff --git a/web/static/js/components/user_list_item.jsx b/web/static/js/components/user_list_item.jsx
@@ -42,7 +42,7 @@ export const UserListItem = ({ user, votes, isVotingStage, currentUser, actions
         </button>
       )}
       <img className={styles.picture} src={imgSrc} alt={identifier} />
-      <p data-hj-masked>{identifier}</p>
+      <p>{identifier}</p>
       { isVotingStage ? (
         <p className={`${styles.allVotesIn} ${allVotesIn ? "opaque" : ""}`}>ALL VOTES IN</p>
       ) : (