edx-platform icon indicating copy to clipboard operation
edx-platform copied to clipboard

[FC-0036] chore: Use openedx_learning from opencraft repo

Open ChrisChV opened this issue 1 year ago • 2 comments

Description

Updates openedx_learning version

Supporting information

  • Github issue: https://github.com/openedx/modular-learning/issues/190
  • Related PRs:
    • https://github.com/openedx/frontend-app-course-authoring/pull/939
    • https://github.com/openedx/openedx-learning/pull/180
  • Internal ticket: FAL-3645

Testing instructions

  • Follow the testing instructions of https://github.com/openedx/frontend-app-course-authoring/pull/939

Other information

  • [ ] Bumb version when https://github.com/openedx/openedx-learning/pull/180 is merged
  • [ ] It is necessary to merge this at the same time of: https://github.com/openedx/frontend-app-course-authoring/pull/939

ChrisChV avatar Apr 08 '24 21:04 ChrisChV

Thanks for the pull request, @ChrisChV! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

  • supporting documentation
  • Open edX discussion forum threads
  • timeline information ("this must be merged by XX date", and why that is)
  • partner information ("this is a course on edx.org")
  • any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

openedx-webhooks avatar Apr 08 '24 21:04 openedx-webhooks

Update: It is necessary to merge this at the same time of: https://github.com/openedx/frontend-app-course-authoring/pull/939

ChrisChV avatar Apr 12 '24 20:04 ChrisChV

LGTM

We need to merge openedx/frontend-app-course-authoring#939 soon after merging this, so waiting until that one is approved and ready to merge.

That PR is approved. Do ping me when this is merged.

xitij2000 avatar Apr 23 '24 08:04 xitij2000

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

openedx-webhooks avatar Apr 25 '24 17:04 openedx-webhooks

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

edx-pipeline-bot avatar Apr 25 '24 19:04 edx-pipeline-bot

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

edx-pipeline-bot avatar Apr 25 '24 19:04 edx-pipeline-bot

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

edx-pipeline-bot avatar Apr 25 '24 19:04 edx-pipeline-bot

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

edx-pipeline-bot avatar Apr 25 '24 19:04 edx-pipeline-bot

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

edx-pipeline-bot avatar Apr 25 '24 20:04 edx-pipeline-bot