Skip to content

Remove xblockutils in favor of xblock.utils in main code#242

Merged
Kelketek merged 3 commits into
masterfrom
fox/remove-xblock-utils
Jun 17, 2026
Merged

Remove xblockutils in favor of xblock.utils in main code#242
Kelketek merged 3 commits into
masterfrom
fox/remove-xblock-utils

Conversation

@Kelketek

@Kelketek Kelketek commented Jun 11, 2026

Copy link
Copy Markdown
Member

Description

This PR removes dependence on the deprecated xblockutils package in main code. It retains usage of the package in testing.

Supporting information

This block is pulling in XBlock-utils into openedx-platform. xblock-utils depends on web-fragments, which is currently being excised as a separate dependency.

Testing instructions

  • Add this as a mount to tutor with tutor mounts add xblock-poll
  • tutor dev stop && tutor dev start -d Restart seems finicky with mounts, so use stop and start.
  • Remove the existing xblock poll by running pip uninstall xblock-poll in the LMS and CMS containers (tutor dev dc exec -it lms -- /bin/bash to enter the LMS, for instance)
  • pip install /mnt/xblock-poll in both the CMS and LMS containers.
  • tutor dev restart
  • Add poll and survey to the advanced modules list of a course.
  • Add both a poll and a survey block
  • Try them out-- verify they seem to function as expected.

Deadline

ASAP

Other information

This change removes support for Quince and Python 3.11.

@Kelketek Kelketek force-pushed the fox/remove-xblock-utils branch from e984bae to 33d50ff Compare June 11, 2026 21:30

@pkulkark pkulkark left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

  • I tested this: verified poll and survey components work as expected after this change.
  • I read through the code.

@Kelketek Kelketek merged commit f711072 into master Jun 17, 2026
5 checks passed
@Kelketek Kelketek deleted the fox/remove-xblock-utils branch June 17, 2026 19:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants