Skip to content

Commit 83c6007

Browse files
authored
feat: remove segment event (#1250)
1 parent a30dccd commit 83c6007

File tree

2 files changed

+0
-12
lines changed

2 files changed

+0
-12
lines changed

src/courseware/course/chat/Chat.jsx

-10
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,6 @@ import PropTypes from 'prop-types';
44
import { Xpert } from '@edx/frontend-lib-learning-assistant';
55
import { injectIntl } from '@edx/frontend-platform/i18n';
66

7-
import { sendTrackEvent } from '@edx/frontend-platform/analytics';
8-
97
const Chat = ({
108
enabled,
119
enrollmentMode,
@@ -42,14 +40,6 @@ const Chat = ({
4240
&& (isEnrolled || isStaff) // display only to enrolled or staff
4341
);
4442

45-
// TODO: Remove this Segment alert. This has been added purely to diagnose whether
46-
// usage issues are as a result of the Xpert toggle button not appearing.
47-
if (shouldDisplayChat) {
48-
sendTrackEvent('edx.ui.lms.learning_assistant.render', {
49-
course_id: courseId,
50-
});
51-
}
52-
5343
return (
5444
<>
5545
{/* Use a portal to ensure that component overlay does not compete with learning MFE styles. */}

src/courseware/course/chat/Chat.test.jsx

-2
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,6 @@ import { initializeMockApp, render, screen } from '../../../setupTest';
88

99
import Chat from './Chat';
1010

11-
jest.mock('@edx/frontend-platform/analytics');
12-
1311
initializeMockApp();
1412

1513
const courseId = 'course-v1:edX+DemoX+Demo_Course';

0 commit comments

Comments
 (0)