Skip to content

feat: New codemods on OpenedX 4 (#28779)#29564

Merged
edx-community-bot merged 1 commit intoopenedx:open-release/maple.masterfrom
cmltaWt0:cmltaWt0/codemods-openedx-4
Dec 15, 2021
Merged

feat: New codemods on OpenedX 4 (#28779)#29564
edx-community-bot merged 1 commit intoopenedx:open-release/maple.masterfrom
cmltaWt0:cmltaWt0/codemods-openedx-4

Conversation

@cmltaWt0
Copy link
Contributor

Description

cherry-picked 7aadf55 from master branch to make changes availabe in maple release.

Supporting information

Relevant JIRA : https://openedx.atlassian.net/browse/BOM-2781
BTR issue: openedx/wg-build-test-release#105

@cmltaWt0 cmltaWt0 requested a review from a team December 13, 2021 10:45
@openedx-webhooks openedx-webhooks added core committer open-source-contribution PR author is not from Axim or 2U waiting on author PR author needs to resolve review requests, answer questions, fix tests, etc. labels Dec 13, 2021
@openedx-webhooks
Copy link

Thanks for the pull request, @cmltaWt0! I've created OSPR-6273 to keep track of it in JIRA.

As a core committer in this repo, you can merge this once the pull request is approved per the core committer reviewer requirements and according to the agreement with your edX Champion.

Copy link

@edx-community-bot edx-community-bot left a comment

Choose a reason for hiding this comment

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

👍

When you're ready to merge, add a comment that says

@edx-community-bot merge

and we'll handle the rest!
CC: @edx/community-engineering @edx/open-release-maintainers

Copy link
Contributor

@nizarmah nizarmah left a comment

Choose a reason for hiding this comment

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

Hello @cmltaWt0! Changes look good to me 👍🏼


Regarding the failed migrations check, I looked at it and it isn't caused by any changes in this pull request.
The same migrations check seems to be failing for some and succeeding for others, it might possibly be resolved by re-running the migrations check.

@cmltaWt0
Copy link
Contributor Author

@edx-community-bot merge

@edx-community-bot edx-community-bot merged commit 376e714 into openedx:open-release/maple.master Dec 15, 2021
@openedx-webhooks
Copy link

@jmbowman: thought you might like to know that cmltaWt0 merged this pull request.

@openedx-webhooks openedx-webhooks added merged and removed waiting on author PR author needs to resolve review requests, answer questions, fix tests, etc. labels Dec 15, 2021
@openedx-webhooks
Copy link

@cmltaWt0 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants