Skip to content

Revert "refactor: move common/lib/capa/capa to xmodule/capa"#30762

Merged
UsamaSadiq merged 1 commit intomasterfrom
revert-30403-iamsobanjaved/BOM-2582-move-common-lib-capa
Jul 21, 2022
Merged

Revert "refactor: move common/lib/capa/capa to xmodule/capa"#30762
UsamaSadiq merged 1 commit intomasterfrom
revert-30403-iamsobanjaved/BOM-2582-move-common-lib-capa

Conversation

@iamsobanjaved
Copy link
Contributor

Reverts #30403

@UsamaSadiq UsamaSadiq marked this pull request as ready for review July 21, 2022 13:20
@UsamaSadiq UsamaSadiq merged commit 4463ee7 into master Jul 21, 2022
@UsamaSadiq UsamaSadiq deleted the revert-30403-iamsobanjaved/BOM-2582-move-common-lib-capa branch July 21, 2022 13:22
@edx-pipeline-bot
Copy link
Contributor

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

@edx-pipeline-bot
Copy link
Contributor

EdX Release Notice: This PR has been deployed to the production environment.

@kdmccormick
Copy link
Member

What happened?

@UsamaSadiq
Copy link
Member

After the deployment on stage, we noticed a high error rate already going on edxapp lms service so as a precaution we reverted the change and decided to merge/deploy the change again next week after the current issue has been resolved.

@kdmccormick
Copy link
Member

Makes sense, thank you!

Add that info to the PR or commit message in the future. It helps those of us that don't have access to edX internal systems.

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.

4 participants