React front end for edX Studio
For an introduction to what this repo is and how it fits into the rest of the edX platform, read Studio-frontend: Developing Frontend Separate from edX Platform.
To install and run locally:
$ git clone email@example.com:edx/studio-frontend.git $ cd studio-frontend $ make up
You can append
-detached to the
make up command to run Docker in the background.
To install a new node package in the repo (assumes container is running):
$ make shell $ npm install <package> --save-dev $ exit $ git add package.json
To make changes to the Docker image locally, modify the Dockerfile as needed and run:
$ docker build -t edxops/studio-frontend:latest .
Webpack will serve pages in development mode at http://localhost:18011.
The following pages are served in the development:
|Edit Image Modal||http://localhost:18011/editImageModal.html|
The development server will run regardless of whether devstack is running along side it. If devstack is not running, requests to the studio API will fail. You can start up the devstack at any time by following the instructions in the devstack repository, and the development server will then be able to communicate with the studio container. API requests will return the following statuses, given your current setup:
|Studio Running?||Logged in?||API return|
|Yes||Yes, non-staff account||403|
|Yes||Yes, staff account||200|
Development Inside Devstack Studio
To load studio-frontend components from the webpack-dev-server inside your studio instance running in Devstack:
- In your devstack edx-platform folder, create
cms/envs/private.pyif it does not exist already.
STUDIO_FRONTEND_CONTAINER_URL = 'http://localhost:18011'to
- Reload your Studio server:
Pages in Studio that have studio-frontend components should now request assets from your studio-frontend docker container's webpack-dev-server. If you make a change to a file that webpack is watching, the Studio page should hot-reload or auto-reload to reflect the changes.
Testing the Production Build Inside Devstack Studio
The Webpack development build of studio-frontend is optimized for speeding up developement, but sometimes it is necessary to make sure that the production build works just like the development build. This is especially important when making changes to the Webpack configs.
Sandboxes use the production webpack build (see section below), but they also take a long time to provision. You can more quickly test the production build in your local docker devstack by following these steps:
- If you have a
cms/envs/private.pyfile in your devstack edx-platform folder, then make sure the line
STUDIO_FRONTEND_CONTAINER_URL = 'http://localhost:18011'is commented out.
- Reload your Studio server:
- Run the production build of studio-frontend by running
make shelland then
npm run buildinside the docker container.
- Copy the production files over to your devstack Studio's static assets
folder by running this make command on your host machine in the
- Run Studio's static asset pipeline:
Your devstack Studio should now be using the production studio-frontend files built by your local checkout.
Testing in studio-frontend
Where do test files go?
If you are developing a component and you are adding new
jsx test files,
the test files would go in the same location as the file. This makes it easier
to track and test. For example, if you are developing a component
src/components/ you would name the test file after the component name
AssetsSearch.test.jsx. Similarly, if you are adding a file
parseDateTime.jsx in the
src/utils/ place the test file at same location with
How to run tests locally
To run the whole suite of tests, you can run
npm run test inside the docker container shell.
make shell npm run test
If you want to run a particular test file only, you can run
npm run test -t <path>.
You can also add
".only" to any
"describe" block in a particular test
file to only run that particular test. For example,
it.only to run only that test
describe.only to run only the tests in that describe block.
How to debug locally running tests
To debug tests running locally, first open the Node debugger:
- navigate to
chrome://inspectin your browser
- choose "Open dedicated DevTools for Node" (it will open in a new window)
- check that the default network configuration in the "Connection" tab is
Next, after adding a
debugger; statement above the test code you'd like to debug, use these commands inside the
make shell node --inspect=0.0.0.0 node_modules/.bin/jest --runInBand
The node debugger should grab focus as soon as your first breakpoint is hit. You can specify individual test files by appending
-- path/to/yourTestFile.test.jsx to the end of the
Testing a Branch on a Sandbox
It is a good practice to test out any major changes to studio-frontend in a sandbox since it is much closer to a production environment than devstack. Once you have a branch of studio-frontend up for review:
Create a new branch in edx-platform off master.
package.jsonin that branch so that it will install studio-frontend from your branch in review:"@edx/studio-frontend": "edx/studio-frontend#your-branch-name",
Commit the change and push your edx-platform branch.
Follow this document on provisioning a sandbox using your edx-platform branch.
The sandbox should automatically pull the studio-frontend branch, run the production webpack build, and then install the dist files into its static assets during provisioning.
This all happens automagically on merges to master, hooray! There are just a few things to keep in mind:
What is the latest version?
Check github, npm, or the npm badge at the top of this README.
package.json no longer contains the correct version (on Github), as it creates an odd loop of "something merged to master, run
semantic-release" -> "
package.json, better check that in and make a PR" -> "a PR merged to master, run
semantic-release", etc. This is the default behavior for
Commit message linting
In order for
semantic-release to determine which release type (major/minor/patch) to make, commits must be formatted as specified by these Angular conventions. TravisBuddy will let you know if anything is wrong before you merge your PR. It can be difficult at first, but eventually you get used to it and the added value of automatic releases is well worth it, in our opinions.
A note on merge messages
Note that when you merge a PR to master (using a merge commit; we've disabled squash-n-merge), there are actually 2 commits that land on the master branch. The first is the one contained in your PR, which has been linted already. The other is the merge commit, which commitlint is smart enough to ignore due to these regexes. The point here is that you should not change the default
Merge pull request <number> from <branch> message on your merge commit, or else the master build will fail and we won't get a deploy.
Updating Latest Docker Image in Docker Hub
If you are making changes to the Dockerfile or docker-compose.yml you may want to include them in the default docker container.
docker tag edxops/studio-frontend:latest edxops/studio-frontend:latest
docker push edxops/studio-frontend:latest
- Check that "Last Updated" was updated here: https://hub.docker.com/r/edxops/studio-frontend/tags/
Adding a new app
There's a bunch of boilerplate that needs to be created to set up a new studio-frontend app that can be independently embedded into a page in Studio. See the openedx-workshop branch, which demonstrates setting up a very basic HelloWorld app.
- Create a new webpack entry
- Add a new HtmlWebpackPlugin to create a new page in the development server to display the component.
- Create a new app root index file which will initialize the app.
- For any new components that the app will use, create a new folder under
src/components/with an upper camel case name.
- To embed the app inside Studio:
CSS in studio-frontend is a bit tricky. Because components are embedded in existing Studio pages, we have to isolate the CSS. This prevents Studio CSS affecting studio-frontend components and from studio-frontend CSS affecting the surrounding Studio page. However, there are a few key points to know about this:
- All studio-frontend styles are scoped to the
- In a way, this div acts like the
<body>element for the embedded studio-frontend component.
- If any elements from studio-frontend are placed outside of this div, then they will be unstyled (or only have Studio styles applied to them).
- In a way, this div acts like the
- Studio-frontend elements are fully reset using a browser default
stylesheet. So, weird
things will occasionally happen with the styling, because it is not a perfect
- Use a browser dev tools style inspector to see what styles are being
applied. Remove styles from
default.cssif you think they might be conflicting with other styling.
- E.g. for some reason, ordered lists appear as unordered. We still have not figured that one out.
- Use a browser dev tools style inspector to see what styles are being applied. Remove styles from
- Selectors that you write in studio-frontend
.scssfiles will be prepended with a selector to the wrapper div during the Webpack build process (
#root.SFE .SFE-wrapper). This is so that studio-frontend styles affect only the contents of the embedded studio-frontend component and so that they are specific enough that they override any Studio styling.
edx-bootstrap.scssfile contains only the Bootstrap variables and mixin definitions. This file is safe to
@importinto individual component
.scssfiles. It allows you to, for example, color an element using the primary color defined in the current Bootstrap theme with the
- Only import
- We currently have CSS modules enabled in the Webpack
css-loader, but aren't
really using the features of it. CSS modules allows you to rename classname
selectors defined in the CSS to be more specific, but we currently have it
configured to leave the names alone. We found it simpler to just reference
Bootstrap classes with a plain string (e.g.
- CSS modules helps avoid class name collision between different components on the same page. We haven't run into this issue with studio-frontend yet, but we might want to consider using it in the future once we do.
Ideally, studio-frontend should not need these CSS hacks. In the future, studio-frontend should control the full HTML page instead of being embedded in a Studio page shell. That way, studio-frontend components would be free from legacy Studio styles and would not need to apply any resets.
If you need assistance with this repository please see our documentation for Getting Help for more information.
We use JIRA for our issue tracker, not GitHub Issues. Please see our documentation for tracking issues for more information on how to track issues that we will be able to respond to and track accurately. Thanks!
How to Contribute
Contributions are very welcome, but for legal reasons, you must submit a signed individual contributor's agreement before we can accept your contribution. See our CONTRIBUTING file for more information -- it also contains guidelines for how to maintain high code quality, which will make your contribution more likely to be accepted.
Reporting Security Issues
Please do not report security issues in public. Please email firstname.lastname@example.org.