Skip to content

chore: remove waffle flags for legacy UI#3

Closed
salman2013 wants to merge 2 commits into
masterfrom
salman/remove-legacy-waffle-flags
Closed

chore: remove waffle flags for legacy UI#3
salman2013 wants to merge 2 commits into
masterfrom
salman/remove-legacy-waffle-flags

Conversation

@salman2013

Copy link
Copy Markdown
Collaborator

Summary

Removes all useNew* waffle flag checks that were gating the legacy Studio UI fallbacks. These flags are no longer needed — the new React-based UI pages are now always used.

Changes

  • Header.tsxgetOutlineLink() always returns /course/${contextId}; isNewHomePage hardcoded to true; removed useWaffleFlags import
  • Breadcrumbs.tsxgetPathToCourseOutlinePage and getPathToCourseUnitPage always resolve to new-UI paths; removed useWaffleFlags and getConfig imports
  • AddComponent.tsx — removed useNewPdfEditor flag; PDF blocks always open the new React editor
  • ChecklistItemBody.jsx — certificate and outline links always point to new-UI routes; removed useWaffleFlags import

Flags removed

useNewCourseOutlinePage, useNewHomePage, useNewUnitPage, useNewPdfEditor, useNewCertificatesPage

Testing

  • Course outline link in header and breadcrumbs navigates to MFE (apps.local.openedx.io:2001)
  • Certificate link in course checklist goes to /course/<id>/certificates
  • PDF block creation opens the React editor modal inline
  • Breadcrumb navigation at section/subsection/unit levels stays within MFE

🤖 Generated with Claude Code

salman2013 and others added 2 commits May 22, 2026 12:50
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@salman2013 salman2013 closed this May 25, 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