From c9b1b966f919871d8736c2015508b9d3b1a9a1c6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?R=C3=A9gis=20Behmo?= Date: Thu, 16 Nov 2023 17:15:26 +0100 Subject: [PATCH] docs: fix README rendering and various issues GitHub could not render the README file because of issues with titles. We take the opportunity to solve a few additional issues. --- README.rst | 39 +++++++++++++++++++-------------------- 1 file changed, 19 insertions(+), 20 deletions(-) diff --git a/README.rst b/README.rst index bd144d4..82ed36e 100644 --- a/README.rst +++ b/README.rst @@ -1,10 +1,10 @@ ########################### frontend-app-learner-record -############################ +########################### |Build Status| |Codecov| |license| -******** +******* Purpose ******* @@ -29,19 +29,20 @@ Developing One Time Setup ============== + .. code-block:: # Clone the repository via SSH git clone git@github.com:openedx/frontend-app-learner-record.git # Enter the directory - cd frontend-app-learner-record + cd frontend-app-learner-record # Clean Install dependencies - npm ci + npm ci # Start the Learner Record MFE - npm start + npm start The page will then be hosted on http://localhost:1990/ @@ -50,27 +51,27 @@ Every time you develop something in this repo .. code-block:: # Grab the latest code - git fetch - git pull + git fetch + git pull # Make a new branch for your changes - git checkout -b / + git checkout -b / # Clean install/update the dev requirements - npm ci + npm ci # Start the Learner Record MFE - npm start + npm start # Using your favorite editor, edit the code to make your change. - vim ... + vim ... # Run the tests after making changes (to verify the status before you make any changes) - npm test + npm test # Commit all your changes - git commit ... - git push + git commit ... + git push # Open a PR and ask for review after the github CI has passed. @@ -81,7 +82,7 @@ Currently, this MFE is not intergrated into the devstack, and must be run locall Credentials requires configuring a Django setting to support directing traffic to the Learner Record MFE: -* ``LEARNER_RECORD_MFE_RECORDS_PAGE_URL`` -- The base URL of the Learne Record MFE +* ``LEARNER_RECORD_MFE_RECORDS_PAGE_URL`` -- The base URL of the Learner Record MFE For more info, see the Learner Records documentation on ReadTheDocs: https://edx-credentials.readthedocs.io/en/latest/learner_records.html. @@ -102,7 +103,7 @@ The Verifiable Credentials UI is a functional addition to the corresponding back Project Structure -================== +================= The source for this project is organized into nested submodules according to the ADR `Feature-based Application Organization `_. @@ -129,9 +130,7 @@ invitation`_, then join our `community Slack workspace`_. For anything non-trivial, the best path is to open an issue in this repository with as many details about the issue you are facing as you -can provide. - -https://github.com/openedx/frontend-app-learner-record/issues +can provide: https://github.com/openedx/frontend-app-learner-record/issues For more information about these options, see the `Getting Help`_ page. @@ -176,7 +175,7 @@ file in this repo. .. _Backstage: https://open-edx-backstage.herokuapp.com/catalog/default/component/{{ cookiecutter.repo_name }} Reporting Security Issues -========================== +========================= Please do not report security issues in public. Please email security@openedx.org.