Skip to content
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

build: Downgrade xblock due to behavior regression. #35244

Merged
merged 1 commit into from
Aug 7, 2024

Conversation

dianakhuang
Copy link
Contributor

@dianakhuang dianakhuang commented Aug 7, 2024

Xblock used to allow overrides of the built-in xblocks. The latest version introduced a regression. See openedx/XBlock#777 for more information

Copy link
Member

@kdmccormick kdmccormick left a comment

Choose a reason for hiding this comment

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

quibble on the wording but LGTM otherwise

requirements/constraints.txt Outdated Show resolved Hide resolved
Xblock used to allow overrides of the built-in xblocks.
The latest version introduced a regression.
@dianakhuang dianakhuang merged commit 35651da into master Aug 7, 2024
49 checks passed
@dianakhuang dianakhuang deleted the diana/pin-xblock branch August 7, 2024 14:58
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

1 similar comment
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

@nsprenkle nsprenkle mentioned this pull request Aug 19, 2024
1 task
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.

3 participants