Skip to content

Conversation

@gabor-boros
Copy link

@gabor-boros gabor-boros commented Oct 4, 2021

Cherry-pick of https://github.com/edx/edx-platform/pull/28411

Imlements OEP-15 by adding two fields to the course settings:

  • Course-wide Custom JS
  • Course-wide Custom CSS
    The resources defined in these fields will be rendered in all course pages.

Rebase b6cb629..0578e1c4c6 onto b6cb629:

  • Add course-wide resources to API for MFE use
  • Revert "Add course-wide resources to API for MFE use" reverts commit 53648dcf0afe3cd171c9dc2eb5e56b871b2bcfb2

Signed-off-by: Gabor Boros gabor.brs@gmail.com

Testing instructions

  1. go to this page
  2. Validate the "listen button" on the right of the course page
  3. Click the button and check it starts reading the text

@gabor-boros gabor-boros changed the title feat: Add course-wide custom scripts [SE-4650] feat: Add course-wide custom scripts Oct 4, 2021
Copy link

@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.

@gabor-boros 👍🏼

  • I tested this by following the testing instructions you described
  • I ensured the cherry-picked commit references the initial commit

@gabor-boros can you please rebase your branch and cherry-pick the commit again using -x so that the reference to the upstream commit is also included?

Imlements OEP-15 by adding two fields to the course settings:
- Course-wide Custom JS
- Course-wide Custom CSS
The resources defined in these fields will be rendered in all course pages.

Rebase b6cb629..0578e1c4c6 onto b6cb629:
- Add course-wide resources to API for MFE use
- Revert "Add course-wide resources to API for MFE use" reverts commit 53648dcf0afe3cd171c9dc2eb5e56b871b2bcfb2

cherry picked from commit e482b85
Signed-off-by: Gabor Boros <gabor.brs@gmail.com>
@gabor-boros gabor-boros force-pushed the gabor/cherry-pick-oep-15 branch from 0003f35 to 1f7bad2 Compare October 4, 2021 18:18
@gabor-boros
Copy link
Author

@nizarmah the commit is amended.

Copy link

@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.

@gabor-boros 👍🏼

  • I tested this by following the testing instructions you described
  • I ensured the cherry-picked commit references the initial commit

@gabor-boros gabor-boros merged commit 5b455de into opencraft-release/koa.3 Oct 4, 2021
@gabor-boros gabor-boros deleted the gabor/cherry-pick-oep-15 branch October 4, 2021 18:31
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