Skip to content

chore: remove transcript editor rollout toggle [LP-1108] - #112

Merged
abhalsod-sonata merged 1 commit into
release-ulmofrom
chore/LP-1108
Sep 4, 2026
Merged

chore: remove transcript editor rollout toggle [LP-1108]#112
abhalsod-sonata merged 1 commit into
release-ulmofrom
chore/LP-1108

Conversation

@abhalsod-sonata

Copy link
Copy Markdown
Member

Removes the temporary rollout toggle gating the in-platform transcript editor, per the product
decision to ship the feature without a flag, and brings the fork in line with the upstream
contribution (openedx#3215).

  • src/data/api.ts: drop the enableTranscriptEditor waffle-flag default.
  • TranscriptMenu.jsx: drop the useWaffleFlags/VideosPageContext imports, the courseId
    context read, the hook call, and the {enableTranscriptEditor && …} wrapper - the
    "Edit transcript" menu item now always renders for videos.
  • TranscriptEditor.scss: one trailing-whitespace fix, matching upstream.

Removal only - no new files, no new dependencies, no test changes needed (no fork test
referenced the flag).

Supporting information

Testing instructions

npm test -- src/files-and-videos

Manual: Course → Videos → open a transcript row's ⋮ menu. "Edit transcript" is present with no
waffle flag configured; the editor opens and saves as before.

Other information

  • Deploy effect - user-visible: this turns the transcript editor on for all course teams
    in Studio on deploy. Previously it was hidden unless contentstore.enable_transcript_editor
    was enabled for the course. This is the intended product outcome (LP-1108).
  • No ordering dependency on the backend PR: if this deploys first, the removed flag is simply
    ignored; if the backend deploys first, the entry stays hidden until this ships.
  • Next parity step (separate PR): port the TypeScript conversion of transcript-editor/
    from upstream feat: add in-platform transcript editor to the videos page openedx/frontend-app-authoring#3215 back to the fork.

@abhalsod-sonata
abhalsod-sonata merged commit cdee978 into release-ulmo Sep 4, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants