Revert "feat: Add audio description functionality to video editor (#91)" (#93)

This reverts commit 0b58258.

Author: viv-helix

src/data/api.ts

@@ -53,7 +53,6 @@ export const waffleFlagDefaults = {
   useNewGroupConfigurationsPage: true,
   useReactMarkdownEditor: true,
   useVideoGalleryFlow: false,
-  enableAudioDescription: false,
 } as const;
 
 export type WaffleFlagName = keyof typeof waffleFlagDefaults;

src/editors/EditorContext.tsx

@@ -15,8 +15,6 @@ export interface EditorContext {
   learningContextId: string;
   /** Is the so-called "Markdown" problem editor available in this learning context? */
   isMarkdownEditorEnabledForContext: boolean;
-  /** Is the audio description upload widget enabled for the current learning context? */
-  isAudioDescriptionEnabled: boolean;
 }
 
 export type EditorContextInit = {
@@ -40,19 +38,15 @@ export const EditorContextProvider: React.FC<{ children: React.ReactNode; } & Ed
   learningContextId,
 }) => {
   const courseIdIfCourse = isCourseKey(learningContextId) ? learningContextId : undefined;
-  const waffleFlags = useWaffleFlags(courseIdIfCourse);
-  const isMarkdownEditorEnabledForContext = waffleFlags.useReactMarkdownEditor;
-  const isAudioDescriptionEnabled = waffleFlags.enableAudioDescription;
+  const isMarkdownEditorEnabledForContext = useWaffleFlags(courseIdIfCourse).useReactMarkdownEditor;
 
   const ctx: EditorContext = React.useMemo(() => ({
     learningContextId,
     isMarkdownEditorEnabledForContext,
-    isAudioDescriptionEnabled,
   }), [
     // Dependencies - make sure we update the context object if any of these values change:
     learningContextId,
     isMarkdownEditorEnabledForContext,
-    isAudioDescriptionEnabled,
   ]);
   return <context.Provider value={ctx}>{children}</context.Provider>;
 };

src/editors/containers/VideoEditor/components/VideoEditorModal.test.tsx

@@ -6,8 +6,6 @@ import { IntlProvider } from '@edx/frontend-platform/i18n';
 import { initializeMockApp } from '@edx/frontend-platform';
 import VideoEditorModal from './VideoEditorModal';
 import { thunkActions } from '../../../data/redux';
-import { EditorContextProvider } from '../../../EditorContext';
-import { mockWaffleFlags } from '../../../../data/apiHooks.mock';
 
 jest.mock('../../../data/redux', () => ({
   ...jest.requireActual('../../../data/redux'),
@@ -20,8 +18,6 @@ jest.mock('../../../data/redux', () => ({
   },
 }));
 
-mockWaffleFlags();
-
 describe('VideoUploader', () => {
   let store;
 
@@ -40,8 +36,6 @@ describe('VideoUploader', () => {
           uploadTranscript: { status: 'inactive' },
           deleteTranscript: { status: 'inactive' },
           fetchVideos: { status: 'inactive' },
-          uploadAudioDescription: { status: 'inactive' },
-          deleteAudioDescription: { status: 'inactive' },
         },
         video: {
           videoSource: '',
@@ -59,7 +53,6 @@ describe('VideoUploader', () => {
             stopTime: '00:00:00',
             total: '00:00:00',
           },
-          audioDescriptionUrl: '',
         },
       },
     });
@@ -76,15 +69,13 @@ describe('VideoUploader', () => {
   const renderComponent = async () => render(
     <AppProvider store={store} wrapWithRouter={false}>
       <IntlProvider locale="en">
-        <EditorContextProvider learningContextId="course-v1:test+test+test">
-          <MemoryRouter
-            initialEntries={[
-              '/some/path?selectedVideoId=id_1&selectedVideoUrl=https://video.com',
-            ]}
-          >
-            <VideoEditorModal isLibrary={false} />
-          </MemoryRouter>
-        </EditorContextProvider>
+        <MemoryRouter
+          initialEntries={[
+            '/some/path?selectedVideoId=id_1&selectedVideoUrl=https://video.com',
+          ]}
+        >
+          <VideoEditorModal isLibrary={false} />
+        </MemoryRouter>
       </IntlProvider>
     </AppProvider>,
   );