-
Notifications
You must be signed in to change notification settings - Fork 197
feat!: migrate to frontend-base #669
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Draft
arbrandes
wants to merge
77
commits into
master
Choose a base branch
from
frontend-base
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from 1 commit
Commits
Show all changes
77 commits
Select commit
Hold shift + click to select a range
89559a4
refactor: migrate to frontend-base
arbrandes 1d29810
refactor: remove/update dotfiles
arbrandes b8f4d49
fix: test environment selection
arbrandes 9b439d7
fix: i18n message export
arbrandes 2045854
fix: make pull_translations
arbrandes 268ccc8
fix: a couple of appId imports
arbrandes c621d58
test: remove unused setupTest line
arbrandes d0051d0
chore: prepare for publication
arbrandes 8175d7e
chore: clean up npmignore
arbrandes b2c6ec2
chore: clean up gitignore
arbrandes d0c27f4
fix: dev site title
arbrandes 6202f7b
feat: handle dashboard role
arbrandes 057b925
feat: Prepare for publication to NPM (#673)
arbrandes 6bed030
chore: bump frontend-base
arbrandes 6f4bf0a
1.0.0-alpha.1
arbrandes dd70abd
chore: bump frontend-base
arbrandes 2509d1b
1.0.0-alpha.2
arbrandes 3adbbbd
fix: turns out react-unit-test-utils was a straight dependency
arbrandes 7f604ba
1.0.0-alpha.3
arbrandes f1d18c4
fix: work around inotify handle starvation
arbrandes a6c5f87
fix: external route roles in dev mode
arbrandes 31581eb
chore: bump frontend-base and regenerate package-lock
arbrandes 117b518
1.0.0-alpha.4
arbrandes 2c295e8
feat!: add design tokens support
arbrandes 0d96fff
build: Update to Node 24
arbrandes 648be5f
fix: missing styles
arbrandes 48eeff4
test: Remove react-unit-test-utils
diana-villalvazo-wgu 21cb518
fix: tests fixed after design tokens support change (#1)
jesusbalderramawgu 4f117df
test: fix minor issues with tests
arbrandes 852444a
1.0.0-alpha.5
arbrandes 35283df
feat: added code spliting on main module
holaontiveros 70576cf
feat!: compile to JS before publishing
arbrandes 0632525
1.0.0-alpha.6
arbrandes d63d9b7
chore: update base route path to match old paths (#797)
holaontiveros 8f0791e
feat: added a generic creditPurchase Url logic (#675)
NoyanAziz 09748bc
feat: improve unenrollment process (#704)
mubbsharanwar a7c5da6
feat: added the ability for instances to use local translations fro…
jajjibhai008 dc10279
fix(deps): remove filesize dependency (#767)
MaxFrank13 3d66fc9
fix: update react-share to v5 (#795)
MaxFrank13 69f04d6
fix(docs): use correct image for custom course banner (#796)
brian-smith-tcril 4607ffe
fix: break circular dependency in site.config.test.tsx
arbrandes ace3a08
refactor: migrate from Redux to React Query
arbrandes fc81a30
fix: add staleTime to prevent excessive refetching
arbrandes fff7b19
fix: remove redundant manual refetch on unenroll
arbrandes cbf7a5b
fix: compute Date.now() per call instead of at module load
arbrandes 07c7083
fix: close modals only after mutation succeeds
arbrandes 9f44855
style: fix lint errors across migrated files
arbrandes 2cbf382
fix: type GlobalDataContext to fix build
arbrandes 97846cd
fix: add @types/jest so ForkTsCheckerWebpackPlugin sees jest globals
arbrandes 909c289
perf: transform course data once in queryFn instead of per card
arbrandes 6a98ddb
refactor: replace BackedDataProvider with React Query cache lookup
arbrandes a0f6d4f
fix: use explicit initializeBase key for query invalidation
arbrandes 7519b96
fix: use smart retry that skips 4xx errors but retries server/network…
arbrandes cb3fa6e
refactor: separate mutation keys from query keys
arbrandes 59ce68e
refactor: replace useReducer with useState in context providers
arbrandes 741fd58
test: add normal user happy path test for useInitializeLearnerHome
arbrandes 9522e27
style: fix lint errors in queryHooks test
arbrandes a932c55
feat: Unenroll survey is configurable through environment variable
marslanabdulrauf 660da94
fix: run tsc-alias after copying assets and limit asset copy to asset…
arbrandes 757064c
fix: address PR #801 review feedback
arbrandes 18f3619
feat: add authenticatedLoader to learner dashboard route
arbrandes 4b636d0
fix: simplify i18n extraction pipeline (#812)
brian-smith-tcril 03f3d7d
feat: support npm workspaces for local development
arbrandes 08b54cd
build: add semantic-release for automated npm publishing
arbrandes a5e861b
feat: add atlasTranslations to package.json
brian-smith-tcril 59a80e6
feat: wire up frontend-base i18n support
brian-smith-tcril eb9a2b1
fix: remove App.messages from app.ts (removed in frontend-base)
brian-smith-tcril c540a57
fix: remove learnerDashboardMessages export from index.ts
brian-smith-tcril 205cc6b
fix: use actual URLs in course action button hrefs
arbrandes 1fa94a4
fix: update tests for link role after href change
arbrandes 3c2ccdc
fix: scope header slot widgets to dashboard role
arbrandes 4a75852
fix: prepend lmsBaseUrl to relative resumeUrl in ResumeButton
arbrandes a1bc648
fix: fix the header
arbrandes 8976fa3
fix: adopt frontend-base shell style manifest
arbrandes ad39800
fix: name learner-dashboard-main chunk in routes
arbrandes 0364a0f
chore: add build:ci smoke test to verify webpack build
arbrandes 56b971c
fix: wrap certPreviewUrl with baseAppUrl in CertificateBanner
arbrandes File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,10 @@ | ||
| /// <reference types="@openedx/frontend-base" /> | ||
|
|
||
| declare module 'site.config' { | ||
| export default SiteConfig; | ||
| } | ||
|
|
||
| declare module '*.svg' { | ||
| const content: string; | ||
| export default content; | ||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,3 @@ | ||
| const { createConfig } = require('@openedx/frontend-base/config'); | ||
|
|
||
| module.exports = createConfig('babel'); |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,22 @@ | ||
| // @ts-check | ||
|
|
||
| const { createLintConfig } = require('@openedx/frontend-base/config'); | ||
|
|
||
| module.exports = createLintConfig( | ||
| { | ||
| files: [ | ||
| 'src/**/*', | ||
| 'site.config.*', | ||
| ], | ||
| }, | ||
| { | ||
| ignores: [ | ||
| 'coverage/*', | ||
| 'dist/*', | ||
| 'documentation/*', | ||
| 'node_modules/*', | ||
| '**/__mocks__/*', | ||
| '**/__snapshots__/*', | ||
| ], | ||
| }, | ||
| ); |
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,18 +1,21 @@ | ||
| const { createConfig } = require('@openedx/frontend-build'); | ||
| const { createConfig } = require('@openedx/frontend-base/config'); | ||
|
|
||
| module.exports = createConfig('jest', { | ||
| module.exports = createConfig('test', { | ||
| setupFilesAfterEnv: [ | ||
| 'jest-expect-message', | ||
| '<rootDir>/src/setupTest.jsx', | ||
| ], | ||
| modulePaths: ['<rootDir>/src/'], | ||
| coveragePathIgnorePatterns: [ | ||
| 'src/segment.js', | ||
| 'src/postcss.config.js', | ||
| 'testUtils', // don't unit test jest mocking tools | ||
| 'src/data/services/lms/fakeData', // don't unit test mock data | ||
| 'src/test', // don't unit test integration test utils | ||
| 'src/__mocks__', | ||
| ], | ||
| moduleNameMapper: { | ||
| '\\.svg$': '<rootDir>/src/__mocks__/svg.js', | ||
| '\\.(jpg|jpeg|png|gif|eot|otf|webp|ttf|woff|woff2|mp4|webm|wav|mp3|m4a|aac|oga)$': '<rootDir>/src/__mocks__/file.js', | ||
| }, | ||
| testTimeout: 120000, | ||
| testEnvironment: 'jsdom', | ||
| }); |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.