When user selects "Other Education" and clicks save - error occurs.
It's caused by the wrong API call payload value `'o'`
Fix: use the `'other'` payload value instead (the valid one).
* refactor: migrated StatusAlert deprecated package to Alert in AgeMessage component
* refactor: migrated StatusAlert deprecated package to Alert for remaining components
* refactor: changed snapshot to unit test to fix ci issue
* fix: updated unit test scrip to make it consistent to timezone differences and fixed snapshot issue
* feat: added unit tests of SocialLinks component to fix ci issue and improve coverage
* refactor: used aria-labelledby in place of ID as suggested in PR
* fix: unnecessary removal of coverage from test script
* fix: typo
* build: adding nvmrc, bumping paragon to latest
Adding .nvmrc is a nicety. Bumping paragon to latest is necessary to ensure compatiblity with the edX override header (frontend-component-header-edx).
* test: snapshot needed updating to account for new data-testid
This is the result of upgrading to the latest paragon. Presumably some test IDs were added.
* build: switch from travis to Github actions.
Subsequent commits will clean up some of the 'npm-dist' code that we're no longer using.
* build: remove npm distribution from repository
Once upon a time, we published the components of the profile MFE to NPM because we believed we might want to embed them in frontend-app-learner-portal. We abandoned that idea about two years ago, but never cleaned up the experiment of publishing the MFE to npm.
This commit does that. The prior commit on this branch disabled it by cutting over from travis to github actions - this commit just finishes the job.
* fix: cleaning up a bad URL for order history.
* docs: cleaning up a documentation URL in openedx.yml
* fix: removing download of Roboto font
None of our other MFEs do this; it was left over from an early design decision. It also doesn't appear to be having any effect, as I think this font is included via Paragon, if at all. It may also be that it's a secondary font choice after something else already available. Regardless, removing these two lines had no effect on the look and feel of the MFE, and reduces bundle size.
* fix: using updated logo instead of pre-rebrand edX logo.
This now uses the FAVICON_URL environment variable like all our other MFEs.
* build: cleanup environment variables
Setting 'null' to an empty string '' (which is falsy and the right thing here)
* fix: clean up unused dependencies
- Removing a number of packages from dependencies that simply were not in use.
- Removing 'font-awesome' dependency which we didn't actually need in our stylesheet - we don't use it, though we were importing it and setting a variable from it. Not sure why it was ever there.
* fix: remove unused dependency
We get 'history' through frontend-platform.
* fix(deps): update font awesome
* test: fixing broken snapshot from font-awesome upgrade
I visually verified the facebook icon is fine by loading up the app.
Co-authored-by: Renovate Bot <bot@renovateapp.com>
Co-authored-by: David Joy <davjoy@gmail.com>
[MICROBA-1321]
* Add two new configuration settings: `ENABLE_LEARNER_RECORD_MFE` and `LEARNER_RECORD_MFE_BASE_URL`
* Add function to `ProfilePage.jsx` to determine where learners should be routed (Credentials or Learner Record MFE) when the `View My Records` button is pressed on the profile page.
* build: upgrades frontend build to v7 and paragon to v16
* test: update test snapshots
Three things happening here:
- The implementation of the external link icon in Hyperlink was updated to have an SVG icon instead of font-awesome
- The implementation of StatefulButton also changed to use SVG icons instead of font-awesome
- Dropdown appears to be setting an onKeyDown handler that it wasn’t before. This is fine.
* fix: fixing broken linter script and linting
The linter script in package.json didn’t specify which files to lint, so it never linted anything. As soon as I fixed the script line, there was suddenly a ton of stuff that needed linting.
This fixes the script and cleans up all the things that needed linting. The vast majority were formatting auto-fixes in VSCode for me.
* fix: setting NODE_ENV to production in .env
Without this change, the NODE_ENV comes through as “null” (a string!) in the app. This causes a number of third party dependencies like React and Redux to potentially go into development mode, slowing them down, or to not realize they’re in production mode, causing them to throw some warnings.
* style: some additional linting
* feat: upgrading to modern paragon and a brand package
This commit updates the app from Paragon 7 to 12 and fixes the breaking changes in between. Mostly small changes to Button and Dropdown, as well as using “container” instead of “container-fluid” to preserve the page’s width as closely as possible.
It also adds the brand package, which is why it’s a feature. Using the brand package allows the MFE to be rebranded by using an npm alias to override the source of the brand.
* test: fixing test snapshots that failed when updating paragon
The test snapshots got a bit out of date when updating paragon. Also removing an unncessary “type” from Dropdown.Toggle which does nothing.
- container has been replaced by container-fluid
- The Button component is a different implementation, which adds slightly different properties to the rendered button. i.e., onKeyDown and disabled, but doesn’t add the id or onBlur.
- The Dropdown doesn’t render its contents until it’s opened, which is why “Upload Photo” and “Remove” are no longer in the snapshot.
-
* build: bumping paragon to 13
* fix: fixing broken test snapshot
btn-outline is definitely not a correct button type.
* fix: using the ‘size’ property on Button
* fix: updating dependencies
We needed to upgrade paragon to 13.1.2 to fix a transpilation issue that was causing ES6 code to be included in the build artifact. All other upgrades here were attempts at fixing that, but they’re all also perfectly valid and good to update, so I left them.
babel-polyfill has been replaced by including core-js and regenerator-runtime.
Upgrading frontend-build fixed an issue with eslint configuration that emerged during the other upgrades.
* fix: switch back to container-fluid
We want to leave it as container-fluid and solve the max width problem through paragon.
* style: cleanup and formatting of SCSS
Also removing an unnecessary variant of primary on a button.
* test: fix broken snapshot test
* fix: bump version of frontend-base to 4.0.0
* fix: bumping a few other versions
* fix: bump footer version and lock travis npm version
* fix: use dist not src for header/footer styless
frontend-base 2.1.x had a bug where segment wasn’t being initialized. Now it is.
Also allowing frontend-base to use its default logging service, which is NewRelicLoggingService.
* Moving files to their new homes.
Subsequent commit will wire everything up again.
* Relink files and split stylesheet.
Requires adding resolve-url-loader to fix sass import relativity.
* Remove browserslist warning.
* Remove the need for ProfileMain
* Fix test issues - needed thunk.
