-
Notifications
You must be signed in to change notification settings - Fork 4.2k
chore: upgrade to XBlock==3.1.0 #34231
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
chore: upgrade to XBlock==3.1.0 #34231
Conversation
c4a90f6 to
b851c97
Compare
b851c97 to
3cc0e78
Compare
|
Sandbox deployment successful 🚀 |
d9f174e to
4ff80a7
Compare
|
Sandbox deployment failed 💥 |
4e92101 to
55b1ef1
Compare
|
Sandbox deployment successful 🚀 |
|
Sandbox deployment successful 🚀 |
dcc240f to
95dcb3e
Compare
|
Sandbox deployment successful 🚀 |
|
Sandbox deployment failed 💥 |
8ba9883 to
cd8e045
Compare
|
Sandbox deployment failed 💥 |
|
Sandbox deployment failed 💥 |
|
Sandbox deployment failed 💥 |
|
Sandbox deployment successful 🚀 |
This has been a no-op for a long time anyway, since HierarchyMixin has been explicitly mixed into XBlock for as long as I remember. Ref: openedx/XBlock#714
633572b to
cef968e
Compare
|
Sandbox deployment successful 🚀 |
|
2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production. |
|
2U Release Notice: This PR has been deployed to the edX production environment. |
1 similar comment
|
2U Release Notice: This PR has been deployed to the edX production environment. |
Description
Completes the upgrade from XBlock 2.0.0 to XBlock 3.1.0, incorporating changes from:
Closes:
Blocked by:
Testing
The XBlock API changes were tested in openedx/XBlock#718 .
For this PR, I'll just use do some end-to-end smoke testing of the studio, author, and learner views of XBlocks in https://github.com/openedx/openedx-demo-course
Note to reviewers: The sandbox is currently failing for reasons unknown (it worked before). Please review anyway; you have my word that before merging, I'll either get the sandbox fixed, or test this out locally.