feat!: migrate to frontend-base by arbrandes · Pull Request #669 · openedx/frontend-app-learner-dashboard

arbrandes · 2025-06-24T18:58:54Z

Description

Refer to the parent issue for a full description.

In short, this refactors the codebase for frontend-base support. There are several breaking changes. It is a long-lived feature branch, and as such, should not be rebased or squashed until time comes for it to either merge to master or become master.

BREAKING CHANGE: Plugins written prior to this will need to be adapted to frontend-base syntax.

BREAKING CHANGE: refactors the MFE for frontend-base.

brian-smith-tcril · 2025-06-24T20:52:57Z

When trying to run npm run dev locally I hit the following error:

node:internal/fs/watchers:247
    const error = new UVException({
                  ^

Error: ENOSPC: System limit for number of file watchers reached, watch '/home/bsmith/code/frontend-app-learner-dashboard/node_modules/serve-index/node_modules/http-errors/LICENSE'

I did a bit of searching and found https://stackoverflow.com/a/65621732

I had the same problem. However, mine was coming from Webpack. Thankfully, they had a great solution on their site:

For some systems, watching many files can result in a lot of CPU or memory usage. It is possible to exclude a huge folder like node_modules using a regular expression:
module.exports = {
  watchOptions: {
    ignored: /node_modules/
  }
};

I haven't tried, but my guess is that adding a watchOptions block to https://github.com/openedx/frontend-base/blob/main/tools/webpack/webpack.config.dev.ts might fix this.

arbrandes · 2025-06-25T11:16:44Z

@brian-smith-tcril, what I did was just increase my fs.inotify.max_user_watches to 524288 in sysctl.conf. There's no reason to stick with the low default on modern systems.

That said, yes, it's arguable that few people will go messing around in node_modules, and fewer still would expect changes to be applied immediately.

brian-smith-tcril · 2025-06-25T16:10:28Z

@brian-smith-tcril, what I did was just increase my fs.inotify.max_user_watches to 524288 in sysctl.conf. There's no reason to stick with the low default on modern systems.

That said, yes, it's arguable that few people will go messing around in node_modules, and fewer still would expect changes to be applied immediately.

Yeah, some of the other answers on that stackoverflow thread recommended increasing that limit. The reason excluding node_modules feels like a better solution to me is that it removes the need to do some "scary" (even if it's a reasonable change) configuration of dev environments.

Update package.json for publication as a "buildless" library.

Bump frontend-base to avoid the inotify handle resource starvation.

@src

Restore accidental reverts from the master port: Dashboard wrapper div and main landmark, test assertions for disabled state and aria-disabled, and initIsPending in Dashboard test. Clean up nits: remove unused React imports, replace escape sequences with literal unicode, simplify ConfirmEmailBanner hooks, fix test names, use @src alias in socialShare, and deduplicate beforeEach mocks in UnenrollConfirmModal tests. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Ensures unauthenticated users are redirected to login before reaching the learner dashboard. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

Add workspaces configuration and workspace-aware scripts for developing with local packages (such as frontend-base). Also, since npm skips bin-linking for workspace packages during install, do it when frontend-base is built. Part of openedx/frontend-base#184 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Adds `.releaserc` and a release GitHub Actions workflow, releasing from the `frontend-base` branch as a prerelease (`alpha`) to the `latest` dist-tag. Co-Authored-By: Claude <noreply@anthropic.com>

openedx-semantic-release-bot · 2026-04-02T18:16:23Z

🎉 This PR is included in version 1.0.0-alpha.7 🎉

The release is available on:

Your semantic-release bot 📦🚀

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

- Rename src/i18n/index.js → index.ts (project uses TypeScript) - Replace placeholder messages object with clean re-export from ./messages - Add src/i18n/messages.d.ts type stub so TypeScript is satisfied before translations:pull has been run - Add translations:pull npm script (backed by openedx translations:pull) - Replace old atlas-based pull_translations Makefile target with npm run translations:pull, passing ATLAS_OPTIONS through - Remove unused intl_imports, i18n, ATLAS_EXTRA_SOURCES, and ATLAS_EXTRA_INTL_IMPORTS Makefile variables - Remove detect_changed_source_translations Makefile target (unused by any CI workflow, extract_translations, or pull_translations) - Add src/i18n/messages.ts, src/i18n/messages/, and src/i18n/site-messages/index.ts to .gitignore (generated files) - Update package-lock to resolve @openedx/frontend-base to 1.0.0-alpha.23 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

openedx-semantic-release-bot · 2026-04-09T13:03:35Z

🎉 This PR is included in version 1.0.0-alpha.8 🎉

The release is available on:

Your semantic-release bot 📦🚀

The old App.messages pattern included apps exporting their messages via \`src/index.ts\`. There is no reason for apps to export this. The file only exists so the app can run as a site with \`npm run dev\`. With frontend-base's i18n pipeline, translations are handled at the site level via \`site.i18n\`. App-only testing didn't catch this because the webpack fallback plugin correctly resolves the missing messages file when \`npm run dev\` is running the app as a site from the repo root (\`src/i18n/\` context). The broken export only surfaces when the compiled \`dist/\` is consumed by a site. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

openedx-semantic-release-bot · 2026-04-09T13:56:14Z

🎉 This PR is included in version 1.0.0-alpha.9 🎉

The release is available on:

Your semantic-release bot 📦🚀

Co-Authored-By: Claude <noreply@anthropic.com>

openedx-semantic-release-bot · 2026-04-13T17:41:07Z

🎉 This PR is included in version 1.0.0-alpha.10 🎉

The release is available on:

Your semantic-release bot 📦🚀

Fixes #829 Co-Authored-By: Claude <noreply@anthropic.com>

openedx-semantic-release-bot · 2026-04-13T17:43:03Z

🎉 This PR is included in version 1.0.0-alpha.11 🎉

The release is available on:

Your semantic-release bot 📦🚀

Co-Authored-By: Claude <noreply@anthropic.com>

openedx-semantic-release-bot · 2026-04-13T18:38:05Z

🎉 This PR is included in version 1.0.0-alpha.12 🎉

The release is available on:

Your semantic-release bot 📦🚀

Use a roles array in the dashboard route so that header slot widgets scope correctly to the dashboard role, and set the Courses link URL accordingly. Also wrap MasqueradeBar in a Paragon Container so its content aligns with the header and main content areas. Co-Authored-By: Claude <noreply@anthropic.com>

openedx-semantic-release-bot · 2026-04-15T16:59:05Z

🎉 This PR is included in version 1.0.0-alpha.13 🎉

The release is available on:

Your semantic-release bot 📦🚀

The shell stylesheet moved from a SCSS file to a JS manifest at @openedx/frontend-base/shell/style. Imports it directly from site.config.dev.tsx and drops the now-redundant site.scss. Removes the ./app.scss subpath export and drops @edx/brand since the dev harness runs unbranded. See openedx/frontend-base#232. Co-Authored-By: Claude <noreply@anthropic.com>

Co-Authored-By: Claude <noreply@anthropic.com>

openedx-semantic-release-bot · 2026-04-22T15:40:08Z

🎉 This PR is included in version 1.0.0-alpha.14 🎉

The release is available on:

Your semantic-release bot 📦🚀

Adds an npm run build:ci script that exercises the real app graph through webpack, and wires it into the CI workflow after the existing build step. Co-Authored-By: Claude <noreply@anthropic.com>

Ports the CertificateBanner portion of 5e69cf8 (#813). The ResumeButton changes from that commit are already covered by 4a75852. Co-Authored-By: Claude <noreply@anthropic.com>

openedx-semantic-release-bot · 2026-04-23T14:40:20Z

🎉 This PR is included in version 1.0.0-alpha.15 🎉

The release is available on:

Your semantic-release bot 📦🚀

refactor: migrate to frontend-base

89559a4

BREAKING CHANGE: refactors the MFE for frontend-base.

arbrandes requested a review from a team as a code owner June 24, 2025 18:58

github-project-automation Bot added this to Aperture-Maintained Jun 24, 2025

arbrandes marked this pull request as draft June 24, 2025 18:59

arbrandes commented Jun 25, 2025

View reviewed changes

Comment thread Makefile Outdated

arbrandes commented Jun 25, 2025

View reviewed changes

Comment thread src/data/services/lms/urls.test.js Outdated

arbrandes mentioned this pull request Jun 25, 2025

Convert Learner Dashboard to frontend-base #657

Open

16 tasks

arbrandes linked an issue Jun 25, 2025 that may be closed by this pull request

Convert Learner Dashboard to frontend-base #657

Open

16 tasks

arbrandes added 19 commits June 25, 2025 17:05

refactor: remove/update dotfiles

1d29810

fix: test environment selection

b8f4d49

fix: i18n message export

9b439d7

fix: make pull_translations

2045854

fix: a couple of appId imports

268ccc8

test: remove unused setupTest line

c621d58

chore: prepare for publication

d0051d0

Update package.json for publication as a "buildless" library.

chore: clean up npmignore

8175d7e

chore: clean up gitignore

b2c6ec2

fix: dev site title

d0c27f4

feat: handle dashboard role

6202f7b

feat: Prepare for publication to NPM (#673)

057b925

chore: bump frontend-base

6bed030

1.0.0-alpha.1

6f4bf0a

chore: bump frontend-base

dd70abd

1.0.0-alpha.2

2509d1b

fix: turns out react-unit-test-utils was a straight dependency

3adbbbd

1.0.0-alpha.3

7f604ba

fix: work around inotify handle starvation

f1d18c4

Bump frontend-base to avoid the inotify handle resource starvation.

arbrandes and others added 5 commits March 23, 2026 16:55

feat: add authenticatedLoader to learner dashboard route

18f3619

Ensures unauthenticated users are redirected to login before reaching the learner dashboard. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

fix: simplify i18n extraction pipeline (#812)

4b636d0

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

build: add semantic-release for automated npm publishing

08b54cd

Adds `.releaserc` and a release GitHub Actions workflow, releasing from the `frontend-base` branch as a prerelease (`alpha`) to the `latest` dist-tag. Co-Authored-By: Claude <noreply@anthropic.com>

arbrandes force-pushed the frontend-base branch from 67dbd29 to 08b54cd Compare April 2, 2026 18:15

openedx-semantic-release-bot added the released on @latest label Apr 2, 2026

brian-smith-tcril and others added 3 commits April 9, 2026 10:02

feat: add atlasTranslations to package.json

a5e861b

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fix: remove App.messages from app.ts (removed in frontend-base)

eb9a2b1

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

arbrandes and others added 2 commits April 13, 2026 14:39

fix: use actual URLs in course action button hrefs

205cc6b

Co-Authored-By: Claude <noreply@anthropic.com>

fix: update tests for link role after href change

1fa94a4

Co-Authored-By: Claude <noreply@anthropic.com>

fix: scope header slot widgets to dashboard role

3c2ccdc

Fixes #829 Co-Authored-By: Claude <noreply@anthropic.com>

fix: prepend lmsBaseUrl to relative resumeUrl in ResumeButton

4a75852

Co-Authored-By: Claude <noreply@anthropic.com>

arbrandes and others added 2 commits April 22, 2026 12:39

fix: name learner-dashboard-main chunk in routes

ad39800

Co-Authored-By: Claude <noreply@anthropic.com>

arbrandes and others added 2 commits April 23, 2026 11:31

chore: add build:ci smoke test to verify webpack build

0364a0f

Adds an npm run build:ci script that exercises the real app graph through webpack, and wires it into the CI workflow after the existing build step. Co-Authored-By: Claude <noreply@anthropic.com>

fix: wrap certPreviewUrl with baseAppUrl in CertificateBanner

56b971c

Ports the CertificateBanner portion of 5e69cf8 (#813). The ResumeButton changes from that commit are already covered by 4a75852. Co-Authored-By: Claude <noreply@anthropic.com>

arbrandes changed the title ~~refactor: migrate to frontend-base~~ feat!: migrate to frontend-base Apr 24, 2026

Conversation

arbrandes commented Jun 24, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Uh oh!

brian-smith-tcril commented Jun 24, 2025

Uh oh!

arbrandes commented Jun 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

brian-smith-tcril commented Jun 25, 2025

Uh oh!

openedx-semantic-release-bot commented Apr 2, 2026

Uh oh!

openedx-semantic-release-bot commented Apr 9, 2026

Uh oh!

openedx-semantic-release-bot commented Apr 9, 2026

Uh oh!

openedx-semantic-release-bot commented Apr 13, 2026

Uh oh!

openedx-semantic-release-bot commented Apr 13, 2026

Uh oh!

openedx-semantic-release-bot commented Apr 13, 2026

Uh oh!

openedx-semantic-release-bot commented Apr 15, 2026

Uh oh!

openedx-semantic-release-bot commented Apr 22, 2026

Uh oh!

openedx-semantic-release-bot commented Apr 23, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

11 participants

arbrandes commented Jun 24, 2025 •

edited

Loading

arbrandes commented Jun 25, 2025 •

edited

Loading