Skip to content

feat: remove enableCourseOptimizerCheckPrevRunLinks flag - #3347

Closed
farhan wants to merge 2 commits into
openedx:masterfrom
farhan:farhan/remove-optimizer-prev-run-links-flag
Closed

farhan wants to merge 2 commits into
openedx:masterfrom
farhan:farhan/remove-optimizer-prev-run-links-flag

Conversation

@farhan

@farhan farhan commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

Note

Stacked on #3346. Until that merges, this PR also shows its commit. Review only the last commit, feat: remove enableCourseOptimizerCheckPrevRunLinks flag.

Description

Removes the enableCourseOptimizerCheckPrevRunLinks waffle flag, so the Course Optimizer always checks for links to the previous course run. Product agreed to keep the feature and drop its rollout flag (decision).

  • Course Author: the Optimizer results now always include the Links to previous course run section, with Update all and per-link update actions. If no such links exist, the section shows "No results found". Before this change, it only showed when contentstore.enable_course_optimizer_check_prev_run_links was on.
  • Operator: the contentstore.enable_course_optimizer_check_prev_run_links flag is no longer read by this MFE. It will be removed from edx-platform in a follow-up PR.

Changes:

  • src/data/api.ts: drop enableCourseOptimizerCheckPrevRunLinks from waffleFlagDefaults.
  • src/optimizer-page/scan-results/ScanResults.tsx: remove the flag checks, and stop passing enabled to useRerunLinkUpdateStatus, so the rerun status query always runs.
  • Tests:
    • Remove the flag mocks and the flag-off tests.
    • "No results found" and "Course updates" can now appear twice (once in broken links and once in previous-run links), so those assertions use getAllByText.

Supporting information

Testing instructions

  1. In Django admin, make sure the contentstore.enable_course_optimizer_check_prev_run_links waffle flag is off or doesn't exist.
  2. Open a course rerun that has links pointing to the original run, then go to Tools → Course Optimizer and run a scan.
  3. The Links to previous course run section should list those links. Click Update all, or update a single link, and check that the links get updated.
  4. Scan a course with no previous-run links. The section should show "No results found".

Other information

  • Follow-up: remove contentstore.enable_course_optimizer_check_prev_run_links and its course_waffle_flags API field from edx-platform.

🤖 Generated with Claude Code

farhan and others added 2 commits November 5, 2026 13:18
Product decided to keep the Course Optimizer as an additive feature
and drop its rollout flag. The Tools menu now shows the Optimizer
link to anyone who can edit course content.

Part of openedx#3090.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Product decided to keep the previous-run links check in the Course
Optimizer and drop its rollout flag. The "Links to previous course
run" section and its rerun status query are now always enabled.

Part of openedx#3090.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@farhan farhan closed this Nov 5, 2026
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.

1 participant