fix: Revert "feat: Integrated Optimizely Prompt Experiment (#52)"

.github/workflows/ci.yml

@@ -1,5 +1,5 @@
 name: Default CI
-on:
+on: 
   push:
     branches:
       - 'main'
@@ -28,7 +28,7 @@ jobs:
     - name: Lint
       run: npm run lint
     - name: Test
-      run: npm run test:ci
+      run: npm run test
     - name: Build
       run: npm run build
     - name: i18n_extract

.github/workflows/release.yml

@@ -22,7 +22,7 @@ jobs:
       - name: Install dependencies
         run: npm ci
       - name: Test
-        run: npm run test:ci
+        run: npm run test
       - name: Release
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

package.json

@@ -19,8 +19,7 @@
     "lint:fix": "fedx-scripts eslint --fix --ext .js --ext .jsx .",
     "snapshot": "fedx-scripts jest --updateSnapshot",
     "start": "fedx-scripts webpack-dev-server --progress",
-    "test": "fedx-scripts jest --coverage --passWithNoTests",
-    "test:ci": "fedx-scripts jest --silent --coverage --passWithNoTests"
+    "test": "fedx-scripts jest --coverage --passWithNoTests"
   },
   "files": [
     "/dist"
@@ -73,7 +72,6 @@
     "@reduxjs/toolkit": "1.8.1",
     "@testing-library/jest-dom": "6.4.5",
     "@testing-library/react": "^12.1.5",
-    "@testing-library/react-hooks": "^8.0.1",
     "@testing-library/user-event": "^14.4.3",
     "glob": "7.2.3",
     "husky": "7.0.4",

src/components/Disclosure/index.jsx

@@ -3,12 +3,10 @@ import React from 'react';
 
 import { Hyperlink, Icon } from '@openedx/paragon';
 import { Chat } from '@openedx/paragon/icons';
-import { ensureConfig, getConfig } from '@edx/frontend-platform/config';
+import { getConfig } from '@edx/frontend-platform/config';
 
 import './Disclosure.scss';
 
-ensureConfig(['PRIVACY_POLICY_URL']);
-
 const Disclosure = ({ children }) => (
   <div className="disclosure d-flex flex-column align-items-stretch px-4 py-3">
     <h2 className="text-light-100">

src/components/Sidebar/index.jsx

@@ -9,15 +9,14 @@ import {
 } from '@openedx/paragon';
 import { Close } from '@openedx/paragon/icons';
 
-import { clearMessages } from '../../data/thunks';
-import { PROMPT_EXPERIMENT_FLAG, PROMPT_EXPERIMENT_KEY } from '../../constants/experiments';
-import { showControlSurvey, showVariationSurvey } from '../../utils/surveyMonkey';
-
 import APIError from '../APIError';
 import ChatBox from '../ChatBox';
 import Disclosure from '../Disclosure';
 import MessageForm from '../MessageForm';
 import './Sidebar.scss';
+import {
+  clearMessages,
+} from '../../data/thunks';
 
 const Sidebar = ({
   courseId,
@@ -29,9 +28,7 @@ const Sidebar = ({
     apiError,
     disclosureAcknowledged,
     messageList,
-    experiments,
   } = useSelector(state => state.learningAssistant);
-  const { variationKey } = experiments?.[PROMPT_EXPERIMENT_FLAG] || {};
   const chatboxContainerRef = useRef(null);
   const dispatch = useDispatch();
 
@@ -72,20 +69,17 @@ const Sidebar = ({
   const handleClick = () => {
     setIsOpen(false);
 
-    if (messageList.length >= 2) {
-      if (variationKey === PROMPT_EXPERIMENT_KEY) {
-        showVariationSurvey();
-      } else {
-        showControlSurvey();
-      }
+    // check to see if hotjar is available, then trigger hotjar event if user has sent and received a message
+    const hasWindow = typeof window !== 'undefined';
+    if (hasWindow && window.hj && messageList.length >= 2) {
+      window.hj('event', 'ocm_learning_assistant_chat_closed');
     }
   };
 
   const handleClearMessages = () => {
     dispatch(clearMessages());
     sendTrackEvent('edx.ui.lms.learning_assistant.clear', {
       course_id: courseId,
-      ...(variationKey ? { experiment_name: PROMPT_EXPERIMENT_FLAG, variation_key: variationKey } : {}),
     });
   };
 
@@ -94,7 +88,7 @@ const Sidebar = ({
   );
 
   const getSidebar = () => (
-    <div className="h-100 d-flex flex-column justify-content-stretch" data-testid="sidebar-xpert">
+    <div className="h-100 d-flex flex-column justify-content-stretch">
       <div className="d-flex flex-column align-items-center p-3">
         <h1 className="font-weight-bold mb-3">
           Hi, I&apos;m Xpert!
@@ -125,7 +119,6 @@ const Sidebar = ({
           aria-label="clear"
           variant="primary"
           type="button"
-          data-testid="sidebar-clear-btn"
         >
           Clear
         </Button>
@@ -137,18 +130,16 @@ const Sidebar = ({
     isOpen && (
       <div
         className="sidebar position-fixed"
-        data-testid="sidebar"
       >
         <IconButton
           className="chat-close position-absolute m-2 border-0"
           src={Close}
           iconAs={Icon}
+          data-testid="close-button"
           onClick={handleClick}
-          alt="close"
           aria-label="close"
           variant="primary"
           invertColors={!disclosureAcknowledged}
-          data-testid="close-button"
         />
         {disclosureAcknowledged ? (getSidebar()) : (<Disclosure>{getMessageForm()}</Disclosure>)}
       </div>