-
Notifications
You must be signed in to change notification settings - Fork 57
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
docs: Release Notes for Quince #441
Conversation
Release notes for the Quince release of OpenEdX platform.
Thanks for the pull request, @feoh! Please note that it may take us up to several weeks or months to complete a review and merge your PR. Feel free to add as much of the following information to the ticket as you can:
All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here. Please let us know once your PR is ready for our review and all tests are green. |
Because my brain runs Markdown whether it's formally correct or not :)
Went through the github changes and found more I think are release note worthy.
@feoh for the convenience of your reviewers, please add a link to the read the docs build for your PR: https://docsopenedxorg--441.org.readthedocs.build/en/441/community/release_notes/quince.html (I found that URL by clicking the "details" link next to the |
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.
The bones are good, but let's flesh it out more.
Because rst and I are like oil and water :)
Because Sphinx / rst is an ornery taskmaster.
Co-authored-by: Peter Pinch <[email protected]>
Co-authored-by: Peter Pinch <[email protected]>
Co-authored-by: Peter Pinch <[email protected]>
Co-authored-by: Peter Pinch <[email protected]>
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.
Looks great! Thanks for your work on this.
I would like to see a couple of late-breaking additions, though.
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.
Can you add a known issue for "Search isn't available in the new Learner Dashboard" and link to openedx/wg-build-test-release#321
@sarina @jmakowski1123 are either of you interested in reviewing this? There's not much time before we need to merge this, but we can always make updates. |
Two related features that are missing are the 1) New Files page, which includes the page as a new MFE with new functionality, and 2) improvements to the video upload flow. I had touched base with @arbrandes on whether the new Files MFE was included, not sure what the outcome of that was? Files release notes: https://docs.google.com/document/d/1tTNCJ54oodT7k38X2YIOpQlk8_gQWFOgypiijJ4jB28/edit#heading=h.tc8j7vnjjwws Video release notes: https://openedx.atlassian.net/wiki/spaces/COMM/pages/3915120666/New+Video+Upload+Flow+Has+Been+Rolled+Out |
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.
Apart from the changes other reviewers already requested, seems good to me! thanks for the awesome work, @feoh
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.
Looks great, thanks!
@feoh 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future. |
Release notes for the Quince release of OpenEdX platform.
Fixes openedx/wg-build-test-release#310