-
Notifications
You must be signed in to change notification settings - Fork 20
Update README to be more helpful #76
Update README to be more helpful #76
Conversation
Thanks for the pull request, @bradenmacdonald! I've created OSPR-4892 to keep track of it in JIRA. As a core committer in this repo, you can merge this once the pull request is approved per the core committer reviewer requirements and according to the agreement with your edX Champion. |
@arbrandes FYI, I'm fixing the blockstore README. This should let you in turn simplify the setup/testing instructions in PRs like openedx-unsupported/frontend-app-library-authoring#4 . BTW I notice that the |
@bradenmacdonald, this is great, thanks! And yes, the MFE README needs a lot of love. I'll handle it when I issue my next PR. 👍 |
@bradenmacdonald Thank you for the contribution. Do you need edX' help with reviewing or merging this? |
@natabene I will be able to review this. 🙂 |
@symbolist Would you be able to review this this sprint? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- I tested the instructions for setting up the devstack.
- I ran the integration tests.
- I checked that the production config sample matches what we are using on one of the CI servers.
@bradenmacdonald 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future. |
@ormsbee, @kdmccormick: thought you might like to know that bradenmacdonald merged this pull request. |
Description
The README is missing a lot of useful information about how to use Blockstore today. This PR brings it up to date.