-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
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.