openedx / openedx/frontend-app-authoring
Follow up Cleanup after removing Legacy UI Waffle Flags
Open
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 17
- Forks
- 218
- Avg merge
- 9d 20h
- Merged PRs (30d)
- 20
Description
Parent ticket: https://github.com/openedx/frontend-app-authoring/issues/3063
Cleanup Tasks
- Remove the
ENABLE_UNIT_PAGEenvironment variable flag and all references to it across the codebase, as the unit page is now the default and this flag is no longer needed. - Remove the
ENABLE_UNIT_PAGE_NEW_DESIGNenvironment variable flag and all references to it, as the new design is now the standard and the flag has been fully superseded. - Remove the
ENABLE_NEW_VIDEO_UPLOAD_PAGEenvironment variable flag and all references to it, as the new video upload page is now the default experience and this toggle is obsolete. - Remove
isNewHomePageprop fromStudioHeader— now thatuseNewHomePagewaffle flag has been removed and the new home page is permanently enabled, theisNewHomePageprop inHeader.tsxis hardcoded totrueand serves no purpose. This requires a two-step change across two repos:@edx/frontend-component-header— remove theisNewHomePageprop fromStudioHeader's interface and destructuring, and hardcodestudioBaseUrl: '/home'directly (currently atStudioHeader.tsx:53).frontend-app-authoring(src/header/Header.tsx:104) — once the above package version is published and consumed, dropisNewHomePagefrom the<StudioHeader>call.
🤖 Generated with Claude Code
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by searching the frontend-app-authoring codebase for ENABLE_UNIT_PAGE, ENABLE_UNIT_PAGE_NEW_DESIGN, ENABLE_NEW_VIDEO_UPLOAD_PAGE, and related references. Review src/header/Header.tsx:104, then coordinate the @edx/frontend-component-header change at StudioHeader.tsx:53 before removing isNewHomePage from the consumer. Done means all three obsolete flags and references are removed and the header prop is removed in both repositories.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- frontend
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100