-
Notifications
You must be signed in to change notification settings - Fork 78
Rewrite OLX documentation #1314
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
Conversation
47fc0e4 to
924ad46
Compare
|
@ormsbee @pdpinch @jswope00 I took a swing at rewriting the OLX documentation - updating screenshots, indicating what's no longer supported in the Studio UI, and making a course in OLX that actually imports into Studio (see openedx/training-courses#68 - that needs to merge before this merges) I don't know how detailed you'd like to be in review. I think at minimum, this is a lot better than what we have now. |
source/educators/references/course_development/awarding_partial_credit.rst
Show resolved
Hide resolved
|
hey, this hasn't fallen off my radar, just has fallen behind some Ulmo-critical libraries stuff. still planning to review soon so that we can get it merged before the holidays. |
|
Thanks. I'd like to get it in before the Ulmo cut if possible, but it shouldn't be terribly difficult to backport if necessary. |
jswope00
left a comment
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.
This is really impressive!! Some changes requested in the comments.
28b4ba6 to
e9c73a6
Compare
|
@jswope00 could you re-review since your review is blocking? |
|
@kdmccormick - I'm going to go ahead and merge this, and we can do a review and make further corrections moving forward, so this can get into the Ulmo release. |
Rewrite OLX documentation to focus on:
Depends on openedx/training-courses#68
Docs: https://docsopenedxorg--1314.org.readthedocs.build/en/1314/educators/navigation/olx.html