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

[abhalsod-sonata]

Description

Removes the temporary rollout toggle for 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#39038) so the two diff as a no-op.

• cms/djangoapps/contentstore/toggles.py: drop ENABLE_TRANSCRIPT_EDITOR
  (contentstore.enable_transcript_editor) and the transcript_editor_enabled() helper.
• rest_api/v1/serializers/course_waffle_flags.py: drop the enable_transcript_editor field.
• rest_api/v1/views/tests/test_course_waffle_flags.py: drop the three flag tests.
• views/tests/test_transcript_settings.py: add test_transcript_upload_handler_returns_200_on_replace.
  This was missing on the fork — feat: Add transcript editor toggle and update related serializers and tests #267's tests only covered the toggle, so the 200-vs-201 upload
  response change had no coverage. Added at the same location as upstream.

transcript_storage_handlers.py (the 200-on-replace change from #267) is unchanged.

Parity check: the fork's net transcript-editor change after this PR (pre-#267 → this branch)
is line-for-line identical to the upstream PR's change set (28 lines each, zero diff).

Supporting information

• Jira: https://2u-internal.atlassian.net/browse/LP-1108
• Upstream PR this brings the fork in line with: feat: return 200 when a transcript upload replaces an existing language openedx/openedx-platform#39038
• Context: Robert's review comment on the upstream PR asking why the test wasn't on the fork
  () and the upstreaming-process Slack thread ().
• Original downstream PR that introduced the toggle: feat: Add transcript editor toggle and update related serializers and tests #267

Testing instructions

pytest cms/djangoapps/contentstore/views/tests/test_transcript_settings.py \
       cms/djangoapps/contentstore/rest_api/v1/views/tests/test_course_waffle_flags.py

Manual: GET /api/contentstore/v1/course_waffle_flags/<course_key> no longer returns
enable_transcript_editor; uploading a transcript for a language that already has one returns 200,
a new language returns 201.

[Copilot]

🟢 Approval recommended

The flag and API surface removal is complete (no remaining references found) and the PR adds targeted test coverage for the 200-vs-201 transcript upload behavior.

Pull request overview

This PR removes the temporary waffle-flag rollout toggle for the Studio in-platform transcript editor, aligning the fork with the upstream implementation by eliminating the flag surface area and associated API/test coverage while adding missing coverage for transcript upload “replace vs create” status codes.

Changes:

• Removed the contentstore.enable_transcript_editor toggle (ENABLE_TRANSCRIPT_EDITOR) and its transcript_editor_enabled() helper.
• Removed the enable_transcript_editor field from the course waffle flags API serializer and removed related view tests.
• Added a regression test ensuring transcript upload returns 200 when replacing an existing transcript (vs 201 when creating a new one).

File summaries

File	Description
cms/djangoapps/contentstore/toggles.py	Removes the transcript editor CourseWaffleFlag and helper method.
cms/djangoapps/contentstore/rest_api/v1/serializers/course_waffle_flags.py	Drops the enable_transcript_editor serializer field/method.
cms/djangoapps/contentstore/rest_api/v1/views/tests/test_course_waffle_flags.py	Removes tests/expected defaults for the dropped flag.
cms/djangoapps/contentstore/views/tests/test_transcript_settings.py	Adds coverage for 200-on-replace transcript upload behavior.

Review details

• Files reviewed: 4/4 changed files
• Comments generated: 0
• Review effort level: Lite

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.